Copilot commented on code in PR #7930:
URL: https://github.com/apache/ignite-3/pull/7930#discussion_r3034040661
##########
modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java:
##########
@@ -301,13 +301,12 @@ public class TableManager implements
IgniteTablesInternal, IgniteComponent {
* @param metricManager Metric manager.
*/
public TableManager(
- String nodeName,
+ InternalClusterNode localNode,
RevisionListenerRegistry registry,
GcConfiguration gcConfig,
ReplicationConfiguration replicationConfiguration,
MessagingService messagingService,
TopologyService topologyService,
- InternalClusterNode localNode,
LockManager lockMgr,
Review Comment:
The TableManager constructor Javadoc is now out of sync with the signature:
it documents `localNode`, `registry`, `gcConfig`, `replicationConfiguration`,
then jumps to `lockMgr`, but the constructor has `MessagingService
messagingService` and `TopologyService topologyService` parameters that are not
documented. Please add the missing `@param messagingService` and `@param
topologyService` entries (and keep the order aligned with the parameter list).
--
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]
