[HBASE-21456] Make WALFactory only used for creating WALProviders

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: HBASE-20952
• Fix Version/s: HBASE-20952
• Component/s: wal
• Hadoop Flags: Reviewed
• Assignee: Ankit Singhal
• Reporter: Josh Elser
• Created: 08/Nov/18 15:56
• Updated: 03/Apr/19 20:48
• Resolved: 03/Apr/19 20:48

Description

As a Factory, WALFactory should only have one job: creating instances of WALProvider.

However, over the years, it has been a landing place for lots of wal-related methods (e.g. constructing readers, WALEntryStream, and more). We want all of this to live in the WALProvider.

We can do this in two steps: have the WALFactory methods invoke the method on the WALProvider (handled elsewhere), then here we can replace usage of the WALFactory "wrapper methods" with the WALProvider itself.