flyrain commented on a change in pull request #4142:
URL: https://github.com/apache/iceberg/pull/4142#discussion_r810418356
##########
File path: core/src/main/java/org/apache/iceberg/ReachableFileUtil.java
##########
@@ -114,4 +119,47 @@ private static TableMetadata
findFirstExistentPreviousMetadata(List<MetadataLogE
}
return manifestListLocations;
}
+
+ /**
+ * Emits a human-readable metadata graph of the current snapshot of the
given table
+ * @param table Iceberg table
+ * @param out output print stream to emit to
+ */
+ public static void printCurrentSnapshot(Table table, PrintStream out) {
Review comment:
I'd suggest to add one more method which accepts snapshot as an
additional parameter, like this. People may also be interested in printing out
a historical snapshot. Most of the code can be reused between these two methods.
```
public static void printSnapshot(Table table, Snapshot snapshot, PrintStream
out) {
```
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
