Add a flag to SnapshotCompletionInterest to indicate the fate of the snapshot.

Here's what I think the contract is. The completion interest will fire on all
finished snapshots, regardless of whether or not the snapshot succeeded, as long
as all nodes have done the work. If node failures happened during the snapshot,
it won't fire.

To determine if the snapshot actually succeeded, check the didSucceed flag in
the event.
Fix a bug in SnapshotSiteProcessor where a failed snapshot could be reported as
succeeded in a very brief window.
ENG-6160: Fail snapshot if data targets for replicated tables fail to close.

It fails the snapshot if the data targets for the partitioned tables fail to
close. Now it does the same for replicated tables.
ENG-6160: Fail the stream snapshot if it fails to send or close.

If the stream snapshot timed out during write or close, it wasn't marking the
snapshot as failed. Now it checks and propagates the exception to the
SnapshotSiteProcessor so that the snapshot can be marked as failed.
Merge pull request #1310 from VoltDB/ENG-6160

ENG-6160: Stream snapshot end failure signaling

r=aweisberg
ENG-6220: and also changed a ack from future log to debug...just not useful for support. (ENG-6212)
For ENG-6209, add logging to give more visiblity into auto-snapshot activity and fix a bug where failed snapshot initiations don't end up invoking an error callback
ENG-6193: Eliminate duplicate parsing code in the CSVLoader.

Consolidated the CSV file parsing code so that there is only one path for
handling errors.

Extracted the data loader from the parser.
Sped up TestCSVLoader by only starting the database once for the whole suite.
Add incorrect column count test to TestCSVLoader.
ENG-6193: Print reason instead of CSV line to the console on error.

The line content will still be logged to the report file along with the reason.
Change timestamp format for voltkvqa client.
ENG-6193: Log reason and line number to console from the CSVLoader.

Instead of logging the line number and the line content to the console, print
the line number and the actual error message so that user knows what's wrong.

The line content is still logged in the report file.
ENG-6193, ENG-6145: Improve CSVLoader error messages.

- Include the column index of the column that caused the parsing error.

- Include actions in most of the parsing error messages.
Merge pull request #1316 from VoltDB/ENG-6209

For ENG-6209, add logging to give more visiblity into auto-snapshot acti...
Merge branch 'master' into ENG-6218-tpcc-perf
V4.3
Merge branch 'master' into ENG-6220
ENG-6193: Addressing Anish's comments.

A bunch of cosmetic changes including method renaming, removing commented dead
code, etc.
Merge pull request #1317 from VoltDB/ENG-6193

ENG-6193: CSVLoader error message improvement.

r=akhanzode