AMashenkov commented on a change in pull request #214:
URL: https://github.com/apache/ignite-3/pull/214#discussion_r669391075
##########
File path: modules/api/src/main/java/org/apache/ignite/table/Table.java
##########
@@ -73,6 +75,16 @@
*/
KeyValueBinaryView kvView();
+ /**
+ * Creates a transactional view of the table.
+ *
+ * @param tx The transaction.
+ * @return Transactional table view.
+ * @see Transaction
+ * @see IgniteTransactions
+ */
+ Table withTx(Transaction tx);
Review comment:
I think we should have this method in all Views (KV and Table) because
user usually reuse cached Table object instance.
kvView.withTx(tx) looks better, than table.withTx(tx).kvView()
--
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]
