apache / carbondata / 714

76%

push

jenkins

[CARBONDATA-2774][BloomDataMap] Exception should be thrown if expression do not satisfy bloomFilter's requirement

If query on string column with bloom index using number as filter value, we would get wrong result. Because DDL will wrap the filter with a datatype conversion, and DatamapChooser chooses datamap by searching column deep down to all children of filter expression. However, bloom filter required the expression is simply column =/in filterValue. So bloom will fail to build any querymodel and no hit blocklet, such that we will get an empty result.

To avoid getting wrong answer silently, we thrown exception instead of only giving a error log currently.

For users who wants to get correct result, can fix the SQL using corresponding data type or disable bloom datamap.

This closes #2545

6 of 6 new or added lines in 1 file covered. (100.0%)

57001 of 70171 relevant lines covered (81.23%)

1.08 hits per line