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 tables. It
seems ok for me to having tableId in PartitionListener, because each listener
is bound to a table. Probably it can be refactored later. For now I would like
to keep it as is.
--
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]
