Aklakan commented on code in PR #3027:
URL: https://github.com/apache/jena/pull/3027#discussion_r2081618510
##########
jena-arq/src/main/java/org/apache/jena/system/Txn.java:
##########
@@ -137,4 +139,46 @@ private static <T extends Transactional> void
onThrowable(Throwable th, T txn) {
txn.end();
} catch (Throwable th2) { th.addSuppressed(th2); }
}
+
+ /**
+ * Begins a transaction and returns a transaction control instance suitable
+ * for use with try-with-resources blocks.
+ * See {@link #begin(Transactional, TxnType)}.
+ */
+ public static AutoTxn begin(Transactional txn, ReadWrite readWrite) {
+ return begin(txn, TxnType.convert(readWrite));
+ }
+
+ /**
+ * Begins a transaction and returns a transaction control instance suitable
+ * for use with try-with-resources blocks.
+ * Allows for raising checked exceptions in an idiomatic way.
+ * Closing the TxnCtl instance will abort the transaction unless it
+ * has been manually committed.
+ * <p>
+ *
+ * Usage example:
+ * <pre>
+ * public void myMethod() throws IOException {
+ * try (AutoTxn txn = Txn.begin(dataset, TxnType.WRITE)) {
+ * // Do work
+ * if (someError) {
+ * throw new IOException();
+ * }
+ * // Must manually call commit on success.
+ * txn.commit();
Review Comment:
When `AutoTxn.close()` is called by the try-with-resources block, then there
seems to be no standard way for close to detect whether it is called after
normal or exceptional block exit. Therefore the logic is that always
exceptional block exit is assumed (-> rollback) and commit() has to be called
manually.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
