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


Reply via email to