[GitHub] carbondata pull request #2654: [CARBONDATA-2896] Adaptive Encoding for Primi...

[manishgupta88]

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

    https://github.com/apache/carbondata/pull/2654#discussion_r214354541
  
    --- Diff: 
processing/src/main/java/org/apache/carbondata/processing/store/TablePage.java 
---
    @@ -363,7 +398,16 @@ public EncodedTablePage getEncodedTablePage() {
               columnPageEncoder = encodingFactory.createEncoder(
                   spec,
                   noDictDimensionPages[noDictIndex]);
    -          encodedPage = 
columnPageEncoder.encode(noDictDimensionPages[noDictIndex++]);
    +          encodedPage = 
columnPageEncoder.encode(noDictDimensionPages[noDictIndex]);
    +          DataType targetDataType =
    +              
columnPageEncoder.getTargetDataType(noDictDimensionPages[noDictIndex]);
    +          if (null != targetDataType) {
    +            LOGGER.info("Encoder result ---> Source data type: " + 
noDictDimensionPages[noDictIndex]
    --- End diff --
    
    make this logger debug and check for debugenabled


---