Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2517#discussion_r205752516
--- Diff:
core/src/main/java/org/apache/carbondata/core/statusmanager/SegmentUpdateStatusManager.java
---
@@ -724,6 +724,7 @@ public void
writeLoadDetailsIntoFile(List<SegmentUpdateDetails> listOfSegmentUpd
brWriter.write(metadataInstance);
} catch (IOException ioe) {
LOG.error("Error message: " + ioe.getLocalizedMessage());
+ fileWrite.setFailed();
--- End diff --
@manishgupta88 Thanks for the review.
Agreed with your point eating exception is not a good idea, handled
throwing back at all the places.
---
