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

    https://github.com/apache/carbondata/pull/2683#discussion_r217322845
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/util/DataTypeUtil.java ---
    @@ -168,6 +168,65 @@ public static Object 
getMeasureObjectBasedOnDataType(ColumnPage measurePage, int
         }
       }
     
    +  /**
    +   * Calculate data percentage in [min, max] scope based on data type
    +   * @param data data to calculate the percentage
    +   * @param min min value
    +   * @param max max value
    +   * @param column column schema including data type
    +   * @return result
    +   */
    +  public static double computePercentage(byte[] data, byte[] min, byte[] 
max, ColumnSchema column) {
    --- End diff --
    
    Bette move this method to CI tool only as it is used only there


---

Reply via email to