SaketaChalamchala commented on code in PR #8981:
URL: https://github.com/apache/ozone/pull/8981#discussion_r2323730733
##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OmSnapshotLocalData.java:
##########
@@ -45,32 +51,33 @@ public abstract class OmSnapshotLocalData {
// Whether SST is filtered
private boolean isSSTFiltered;
- // Map of Table to uncompacted SST file list on snapshot create
- private Map<String, Set<String>> uncompactedSSTFileList;
-
// Time of last compaction, in epoch milliseconds
private long lastCompactionTime;
// Whether the snapshot needs compaction
private boolean needsCompaction;
- // Map of version to compacted SST file list
- // Map<version, Map<Table, sstFileList>>
- private Map<Integer, Map<String, Set<String>>> compactedSSTFileList;
+ private UUID previousSnapshotId;
+
+ // Map of version to VersionMeta, using linkedHashMap since the order of the
map needs to be deterministic for
+ // checksum computation.
+ private final LinkedHashMap<Integer, VersionMeta> versionSstFileInfos;
public static final Charset CHARSET_ENCODING = StandardCharsets.UTF_8;
private static final String DUMMY_CHECKSUM = new String(new byte[64],
CHARSET_ENCODING);
/**
* Creates a OmSnapshotLocalData object with default values.
*/
- public OmSnapshotLocalData(Map<String, Set<String>> uncompactedSSTFileList) {
+ public OmSnapshotLocalData(List<LiveFileMetaData> uncompactedSSTFileList,
UUID previousSnapshotId) {
Review Comment:
What is the reason to store previousSnapshotId in the snapshot YAML file
when it is already available in the SnapshotInfoTable?
Do we plan to update the previous snapshot ID whenever there is a change to
the snapshot chain i.e., snapshot delete?
--
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]
