nihal0107 commented on a change in pull request #3921: URL: https://github.com/apache/carbondata/pull/3921#discussion_r488461592
########## File path: processing/src/main/java/org/apache/carbondata/processing/loading/converter/impl/RowConverterImpl.java ########## @@ -119,10 +118,6 @@ public CarbonRow convert(CarbonRow row) throws CarbonDataLoadingException { .getTableProperties(); String spatialProperty = properties.get(CarbonCommonConstants.SPATIAL_INDEX); boolean isSpatialColumn = false; - Object[] rawData = row.getRawData(); - if (rawData == null) { - rawData = row.getData() == null ? null : row.getData().clone(); Review comment: I had only added this scenario last time because at that time we wanted rawdata for every bad record action. But now we need this only in case of bad record logger is enable or action is redirect. And in this case, rawdata will be always available. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org