vldpyatkov commented on a change in pull request #399:
URL: https://github.com/apache/ignite-3/pull/399#discussion_r733163302



##########
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();
+
+    /**
+     * Updates internal table raft group service for given partition.
+     *
+     * @param p Partition.
+     * @param raftGrpSvc Raft group service.
+     */
+    void updateInternalTableRaftGroupService(int p, RaftGroupService 
raftGrpSvc);

Review comment:
       This method was here before, I moved it to the interface.
   Where is a violation here?




-- 
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]


Reply via email to