kunal642 commented on a change in pull request #3680: [CARBONDATA-3766] Fixed
desc formatted and show segment data size issues
URL: https://github.com/apache/carbondata/pull/3680#discussion_r404617995
##########
File path:
core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java
##########
@@ -527,10 +527,11 @@ public static boolean updateTableStatusFile(CarbonTable
carbonTable, String segm
for (LoadMetadataDetails detail : listOfLoadFolderDetailsArray) {
// if the segments is in the list of marked for delete then update
the status.
if (segmentId.equals(detail.getLoadName())) {
+ detail.setLoadEndTime(System.currentTimeMillis());
Review comment:
I think we should not add external segments check for common things, it
should be okay if we set here because we are not using this end time in any
other logic/decision making
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
