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

    https://github.com/apache/carbondata/pull/1297#discussion_r138916072
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/datastore/page/LazyColumnPage.java
 ---
    @@ -91,9 +93,24 @@ public float getFloat(int rowId) {
         throw new UnsupportedOperationException("internal error");
       }
     
    -  @Override
    -  public BigDecimal getDecimal(int rowId) {
    -    return columnPage.getDecimal(rowId);
    +  @Override public BigDecimal getDecimal(int rowId) {
    --- End diff --
    
    Can you move all @Override to previous line


---

Reply via email to