[HBASE-21250] Refactor WALProcedureStore and add more comments for better understanding the implementation

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.0.0-alpha-1, 2.2.0, 2.1.1, 2.0.3
• Component/s: proc-v2
• Hadoop Flags: Reviewed
• Assignee: Duo Zhang
• Reporter: Duo Zhang
• Created: 28/Sep/18 09:56
• Updated: 08/Oct/18 18:13
• Resolved: 07/Oct/18 09:36

Description

The implementation is complicated and lack of comments to say how it works.

/**
 * WAL implementation of the ProcedureStore.
 * @see ProcedureWALPrettyPrinter for printing content of a single WAL.
 * @see #main(String[]) to parse a directory of MasterWALProcs.
 */

I think at least we can move sub classes to separated files to make the class smaller, and add more comments to describe what is going on here.

Issue Links

• links to: Review Board