ascherbakoff commented on a change in pull request #400:
URL: https://github.com/apache/ignite-3/pull/400#discussion_r749305598



##########
File path: 
modules/table/src/main/java/org/apache/ignite/internal/table/distributed/raft/PartitionListener.java
##########
@@ -77,19 +74,28 @@
  * Partition command handler.
  */
 public class PartitionListener implements RaftGroupListener {
-    /** Partition storage. */
-    private final PartitionStorage storage;
+    /** Table ID. */
+    private final IgniteUuid tableId;

Review comment:
       Because tx manager requires a table id for acquring locks for table 
keys. It seems ok for me to having tableId in PartitionListener, because each 
listener is bound to a table. I've refactored it to be less confusing.




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