Thanks for the review Andrew Wang and Lei (Eddy) Xu.
Adding an option for including node details on maintenance state does sound like a good idea and helps in making the patch backward compatible. Attached v03 patch, to address the following
- Added an option -includeMaintenanceState to fsck command to print maintenance state related node details
- Updated HDFSCommands.md to include help for the new option in fsck command
- Updated TestFsck to verify fsck results with and without the new command option
PS: Patch might have checkstyle issues w.r.t function length, and on non private variables. All these are already existing ones and can be fixed outside of this bug.
Lei (Eddy) Xu, can you please review the patch ? Once I hear your comments on this, can extend the patch to branch-2 as well. Please let me know.