kevinrr888 opened a new pull request, #5670:
URL: https://github.com/apache/accumulo/pull/5670
Avoids creating multiple conditional writers for UserFateStore.
* FateStore now extends AutoCloseable
* MetaFateStore.close() falls back to AbstractFateStores impl of
close() which is no-op
* UserFateStore.close() closes the conditional writer
* All tests using a FateStore now call close()
* The real fate stores are closed on the shutdown of fate
* UserFateStore now only creates a single conditional writer
* Num threads for this conditional writer set by a new prop
CONDITIONAL_WRITER_THREADS_MAX_FATE_TABLE
* writer in UserFateStore is a memoized Supplier so it's not created
when using a ReadOnlyFateStore.
Diff is much larger than actual changes, much of the changes are from
needing to put the fate stores in a try-with-resources.
Verified sunny passes and all FATE tests pass
closes #5660
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
