ctubbsii commented on issue #1124: URL: https://github.com/apache/fluo/issues/1124#issuecomment-1318015520
Note to potential contributors: this could be done in two smaller steps, as separate PRs. 1. inline all instances of `TestTransaction.done()` (and remove the method), and change `implements TransactionBase` to `implements Transaction` (adding `@Override` annotations to `close()` and `commit()` would be nice, too), and 2. track down `TestTransaction` declarations and convert them to use try-with-resources blocks instead of explicitly calling `close()` -- 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: notifications-unsubscr...@fluo.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
