guihecheng commented on code in PR #3398:
URL: https://github.com/apache/ozone/pull/3398#discussion_r871194816
##########
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/keyvalue/TarContainerPacker.java:
##########
@@ -213,6 +221,31 @@ private byte[] readEntry(InputStream input, final long
size)
return output.toByteArray();
}
+ private void includeMetadataFiles(KeyValueContainerData containerData,
+ ArchiveOutputStream archiveOutput) throws IOException {
+ if (containerData.getSchemaVersion().equals(OzoneConsts.SCHEMA_V3)) {
+ includeDumpFiles(Paths.get(containerData.getMetadataPath()),
Review Comment:
Continue the last comment here.
We have a concrete on-disk db directory under the `metadata` directory(like:
metadata/1-dn-container.db) for per-container db instance, but we don't have
one dedicated for temp sst files, for now we just put it alongside with the
container meta file(like: metadata/{1-container, block_data.data,
metadata.data, ...}).
So I intend to include exactly the files that match the suffix here.
--
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]
