Github user kevinjmh commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2879#discussion_r229287888
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/loading/steps/CarbonRowDataWriterProcessorStepImpl.java
---
@@ -308,7 +312,7 @@ private void processBatch(CarbonRowBatch batch,
CarbonFactHandler dataHandler, i
}
writeCounter[iteratorIndex] += batch.getSize();
} catch (Exception e) {
- throw new CarbonDataLoadingException("unable to generate the mdkey",
e);
+ throw new CarbonDataLoadingException(e);
--- End diff --
The KeyGenException extend Exception, it needs
CarbonDataLoadingException(RuntimeException) to wrap and throw.
---
