Skip to content

Commit

Permalink
[Vectorized](compile) Fix compile error and warning (#7780)
Browse files Browse the repository at this point in the history
  • Loading branch information
zenoyang authored Jan 17, 2022
1 parent c86d691 commit 66b3b1d
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
1 change: 1 addition & 0 deletions be/src/vec/columns/column.h
Original file line number Diff line number Diff line change
Expand Up @@ -220,6 +220,7 @@ class IColumn : public COW<IColumn> {
*/
virtual Ptr filter_by_selector(const uint16_t* sel, size_t sel_size, Ptr* ptr = nullptr) {
LOG(FATAL) << "column not support filter_by_selector";
__builtin_unreachable();
};

/// Permutes elements using specified permutation. Is used in sortings.
Expand Down
3 changes: 3 additions & 0 deletions be/src/vec/functions/function.h
Original file line number Diff line number Diff line change
Expand Up @@ -404,6 +404,7 @@ class IFunction : public std::enable_shared_from_this<IFunction>,
const ColumnNumbers& /*arguments*/,
size_t /*result*/) const final {
LOG(FATAL) << "prepare is not implemented for IFunction";
__builtin_unreachable();
}

Status prepare(FunctionContext* context, FunctionContext::FunctionStateScope scope) override {
Expand All @@ -412,10 +413,12 @@ class IFunction : public std::enable_shared_from_this<IFunction>,

[[noreturn]] const DataTypes& get_argument_types() const final {
LOG(FATAL) << "get_argument_types is not implemented for IFunction";
__builtin_unreachable();
}

[[noreturn]] const DataTypePtr& get_return_type() const final {
LOG(FATAL) << "get_return_type is not implemented for IFunction";
__builtin_unreachable();
}

protected:
Expand Down
4 changes: 2 additions & 2 deletions be/src/vec/functions/function_grouping.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,11 @@
// specific language governing permissions and limitations
// under the License.

#include "function_grouping.h"
#include "vec/functions/function_grouping.h"

namespace doris::vectorized {
void register_function_grouping(SimpleFunctionFactory& factory) {
factory.register_function<FunctionGrouping>();
factory.register_function<FunctionGroupingId>();
}
}
} // namespace doris::vectorized

0 comments on commit 66b3b1d

Please sign in to comment.