sanpwc commented on a change in pull request #399:
URL: https://github.com/apache/ignite-3/pull/399#discussion_r732484630
##########
File path:
modules/table/src/main/java/org/apache/ignite/internal/table/InternalTable.java
##########
@@ -33,7 +34,22 @@
* Internal table facade provides low-level methods for table operations.
* The facade hides TX/replication protocol over table storage abstractions.
*/
-public interface InternalTable {
+public interface InternalTable extends AutoCloseable {
+ /**
+ * Gets a storage for the table.
+ *
+ * @return Table storage.
+ */
+ @NotNull TableStorage storage();
Review comment:
Why we need it here? How it's related to TableManager stop?
It's declared that InternalTable is an interface that **provides low-level
methods for table operations**, do you believe that storage is a table
operation?
--
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]
