Details
-
Task
-
Status: Closed
-
Major
-
Resolution: Fixed
-
None
-
None
Description
Current logic used in migrating archives in SegmentStoreMigrator silently swallows exceptions coming from the segment, binary refs and graph migration methods:
try (SegmentArchiveReader reader = sourceManager.forceOpen(archiveName)) { SegmentArchiveWriter writer = targetManager.create(archiveName); try { migrateSegments(reader, writer); migrateBinaryRef(reader, writer); migrateGraph(reader, writer); } finally { writer.close(); } }
Adding a catch block which logs the possible exceptions would make analysing logs easier.