[GitHub] carbondata pull request #2142: [CARBONDATA-2316] Executor task is failed but...

[manishgupta88]

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

    https://github.com/apache/carbondata/pull/2142#discussion_r180646852
  
    --- Diff: 
processing/src/main/java/org/apache/carbondata/processing/merger/CompactionResultSortProcessor.java
 ---
    @@ -165,8 +168,12 @@ public boolean execute(List<RawResultIterator> 
resultIteratorList) {
             readAndLoadDataFromSortTempFiles();
           }
           isCompactionSuccess = true;
    -    } catch (Exception e) {
    +    } catch (BadRecordFoundException e) {
    --- End diff --
    
    BadRecordFoundException  will never come in case of compaction so remove 
this catch block


---