wumeibanfa commented on code in PR #56600:
URL: https://github.com/apache/doris/pull/56600#discussion_r2390096608
##########
be/src/vec/functions/function_bitmap.cpp:
##########
@@ -940,31 +937,28 @@ struct BitmapHasAll {
using TData = std::vector<BitmapValue>;
using ResTData = typename ColumnUInt8::Container;
- static void vector_vector(const TData& lvec, const TData& rvec, ResTData&
res) {
+ static void vector_vector(TData lvec, const TData& rvec, ResTData& res) {
size_t size = lvec.size();
for (size_t i = 0; i < size; ++i) {
uint64_t lhs_cardinality = lvec[i].cardinality();
- auto bitmap = const_cast<BitmapValue&>(lvec[i]);
- bitmap |= rvec[i];
- res[i] = bitmap.cardinality() == lhs_cardinality;
+ lvec[i] |= rvec[i];
+ res[i] = lvec[i].cardinality() == lhs_cardinality;
}
}
- static void vector_scalar(const TData& lvec, const BitmapValue& rval,
ResTData& res) {
+ static void vector_scalar(TData lvec, const BitmapValue& rval, ResTData&
res) {
Review Comment:
因为这里使用了const&，又使用const_cast修改了lvec
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
