You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@carbondata.apache.org by jackylk <gi...@git.apache.org> on 2017/09/06 14:16:42 UTC

[GitHub] carbondata pull request #1311: [CARBONDATA-1439] Wrong Error message shown f...

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?


---