Github user kumarvishal09 commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1346#discussion_r138254259
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/scan/filter/executer/RowLevelRangeLessThanFiterExecuterImpl.java
 ---
    @@ -267,11 +267,15 @@ private BitSet 
getFilteredIndexes(DimensionColumnDataChunk dimensionColumnDataCh
           DirectDictionaryGenerator directDictionaryGenerator = 
DirectDictionaryKeyGeneratorFactory
    --- End diff --
    
    Now as we are not adding one in default value, while searching for last 
index of null value in binary search, we need to add +1 in search result as 
search will be last index of null value inclusive


---

Reply via email to