Sammi Chen
Sammi Chen
@adoroszlai @bshashikant for the code review. New commits are uploaded to address the comments. The failed integration test is about TestSCMInstallSnapshot, which is not relevant.
Thanks @ArafatKhan2198 , can you tried other fs/CLI commands, to see if their WARNING messages are resolved by this PR?
Thanks @symious to improve the replication feature. Agree with @errose28 suggestions. We can create a proper sub-directory under each "hdds.datanode.dir" for downloaded containers, instead of introduce a new "hdds.datanode.replication.work.dir" configuration,...
> > * IMO `hdds.datanode.replication.work.dir` should be deprecated and ignored. Datanodes require a clean disk state in their container directories on restart (see [HDDS-6441](https://issues.apache.org/jira/browse/HDDS-6441) and [HDDS-6449](https://issues.apache.org/jira/browse/HDDS-6449)), so any non-atomic FS...
@errose28 thanks for the info about how the downgrade should be verified. Right, it makes sense to deprecate hdds.datanode.replication.work.dir considering all its known issues. Hey @symious, could you update the...
Hey @jojochuang, could you rebase this PR? I can help with the review.
Hey @kuenishi , I left one comment on JIAR HDDS-5905.
Hi @kerneltime , would you like to take a look of this PR?
The last patch LGTM + 1. Thanks @sumitagrawl for the feature improvement, and @adoroszlai for the code review.
Hey @Xushaohong , could you past some example output of this command? For both console output and file output.