Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1311#discussion_r137280154
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/newflow/converter/impl/RowConverterImpl.java
---
@@ -158,8 +158,10 @@ public CarbonRow convert(CarbonRow row) throws
CarbonDataLoadingException {
if (!logHolder.isLogged() && logHolder.isBadRecordNotAdded()) {
badRecordLogger.addBadRecordsToBuilder(copy.getData(),
logHolder.getReason());
if (badRecordLogger.isDataLoadFail()) {
- String error = "Data load failed due to bad record: " +
logHolder.getReason() +
--- End diff --
What is wrong with this message?
---