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

    https://github.com/apache/carbondata/pull/2879#discussion_r229280564
  
    --- Diff: 
processing/src/main/java/org/apache/carbondata/processing/loading/steps/DataWriterProcessorStepImpl.java
 ---
    @@ -259,7 +259,7 @@ public void processRow(CarbonRow row, CarbonFactHandler 
dataHandler) throws KeyG
           readCounter++;
           dataHandler.addDataToStore(row);
         } catch (Exception e) {
    -      throw new CarbonDataLoadingException("unable to generate the mdkey", 
e);
    --- End diff --
    
    no need to wrap the exception, just remove the try-catch code


---

Reply via email to