ACCUMULO-3249 Some extra logging that's missing to get a clear picture.
ACCUMULO-3249 Clarify some comments
ACCUMULO-3249 Remove an unnecessary utility method for clarity and reduce visibility on some internal methods

A number of the methods in ReplicationTableUtil shouldn't be directly accessed
by clients but were still public or protected. Make things a little more specific
by making them package private instead (to still allow for tests to work).
ACCUMULO-3249 When recovery of a tablet doesn't recover anything from a WAL, we must ignore it.

Marking a WAL as closed when recovery doesn't use any mutations from the WAL
is an incorrect assertion. Even though the current Tablet didn't find any
mutations from this WAL doesn't mean that other Tablets also won't find
any mutations. Even worse, other Tablets could *continue to use* this
WAL for new data.
ACCUMULO-3249 Extra comments about issuing updates to replication status for WALs.
ACCUMULO-3249 Work should be created regardless of open or closed
ACCUMULO-3249 Create work only when there is work to be done for the status.
ACCUMULO-3249 New IT which ensures that closed status msgs aren't written for wals with no records for the tablet
ACCUMULO-3249 Didn't wait long enough to check for expected condition