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

Posted to issues@carbondata.apache.org by shivangi1015 <gi...@git.apache.org> on 2017/12/15 07:34:13 UTC

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

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

    https://github.com/apache/carbondata/pull/1311#discussion_r157137865
  
    --- 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 --
    
    Even when the BAD_RECORDS_ENABLE is set to true, the error message includes this: enable the bad record logger to know the detail reason. 


---