kangkaisen commented on pull request #3571: URL: https://github.com/apache/incubator-doris/pull/3571#issuecomment-631204907
@EmmyMiao87 Hi. 当前对于 bitmap_intersect null的处理是不合理的。 按照SQL标准，sum(null), min(null), max(null)结果为 null. count(null)为0. 所以 对于 bitmap_intersect，bitmap_union 返回bitmap的函数，输入全是null时，应该返回null, 对于 bitmap_intersect_count, bitmap_union_count 等返回count值的函数， 输入全是null时，应该返回0. 在之前最开始的时候，我认为所有hll,bitmap的函数都不需要返回null，对于输入是null，我们返回empty bitmap 或者empty hll 即可。（it's my fault...） 所以目前bitmap_init, bitmap_serialize, bitmap_finalize的实现都是not nullable的。 所以你这个PR，你应该实现新的函数 nullable_bitmap_int, nullable_bitmap_serialize。nullable_bitmap_serialize 中src是null, 应该返回null，而不是空的empty. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
