[GitHub] carbondata pull request #1593: [CARBONDATA-1837] Reusing origin row to reduc...

[jackylk]

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

    https://github.com/apache/carbondata/pull/1593#discussion_r155001692
  
    --- Diff: 
processing/src/main/java/org/apache/carbondata/processing/loading/row/CarbonRowBatch.java
 ---
    @@ -61,4 +61,19 @@ public CarbonRow next() throws NoSuchElementException {
       @Override public void remove() {
     
       }
    +
    +  /**
    +   * set current row, this can be used to set value for the RowBatch while 
iterating it
    +   * @param row
    +   */
    +  public void setCurrentRow(CarbonRow row) {
    +    rowBatch[index - 1] = row;
    --- End diff --
    
    why not use index, but index - 1?


---