carbondata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From QiangCai <...@git.apache.org>
Subject [GitHub] carbondata pull request #1772: [CARBONDATA-1995] Unify all writer steps and ...
Date Thu, 01 Feb 2018 08:38:47 GMT
Github user QiangCai commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/1772#discussion_r165287316
  
    --- Diff: processing/src/main/java/org/apache/carbondata/processing/loading/steps/WriterProcessorStepImpl.java
---
    @@ -271,37 +262,18 @@ private CarbonRow convertRow(CarbonRow row) throws KeyGenException
{
       }
     
       private void processBatch(CarbonRowBatch batch, CarbonFactHandler dataHandler, int
iteratorIndex)
    -      throws CarbonDataLoadingException {
    -    try {
    -      while (batch.hasNext()) {
    -        CarbonRow row = batch.next();
    -        CarbonRow converted = convertRow(row);
    -        dataHandler.addDataToStore(converted);
    -        readCounter[iteratorIndex]++;
    +  {
    +    while (batch.hasNext()) {
    +      CarbonRow row;
    +      if (noSort) {
    --- End diff --
    
    move 'if' statement to the outside of 'while' loop


---

Mime
View raw message