[GitHub] [ignite-3] AMashenkov commented on a diff in pull request #2140: [IGNITE-19587] Sql. Remove execution-related part from IgniteTable.

["AMashenkov (via GitHub)" <gi...@apache.org>]

AMashenkov commented on code in PR #2140:
URL: https://github.com/apache/ignite-3/pull/2140#discussion_r1226776026


##########
modules/core/src/main/java/org/apache/ignite/internal/thread/StripedThreadPoolExecutor.java:
##########
@@ -105,7 +106,7 @@ public void execute(Runnable cmd) {
      * @throws NullPointerException       if the task is {@code null}.
      */
     public CompletableFuture<?> submit(Runnable task, int idx) {
-        return CompletableFuture.runAsync(task, execs[threadId(idx)]);
+        return CompletableFuture.runAsync(task, commandExecutor(idx));

Review Comment:
   `@param idx` is missed in javadoc.
   Would you please fix this?



-- 
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: notifications-unsubscribe@ignite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org