[GitHub] [ignite-3] tkalkirill commented on a diff in pull request #1233: IGNITE-17894 Implement RAFT snapshot streaming receiver

[GitBox <gi...@apache.org>]

tkalkirill commented on code in PR #1233:
URL: https://github.com/apache/ignite-3/pull/1233#discussion_r1002999251


##########
modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java:
##########
@@ -404,6 +408,15 @@ public TableManager(
                 TimeUnit.MILLISECONDS,
                 new LinkedBlockingQueue<>(),
                 NamedThreadFactory.create(nodeName, "tableManager-io", LOG));
+
+        incomingSnapshotsExecutor = new ThreadPoolExecutor(

Review Comment:
   I looked at the methods in **Executors**, it seems to me that they do not quite suit me, I changed the pool configuration a little.



##########
modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java:
##########
@@ -404,6 +408,15 @@ public TableManager(
                 TimeUnit.MILLISECONDS,
                 new LinkedBlockingQueue<>(),
                 NamedThreadFactory.create(nodeName, "tableManager-io", LOG));
+
+        incomingSnapshotsExecutor = new ThreadPoolExecutor(

Review Comment:
   I looked at the methods in `Executors`, it seems to me that they do not quite suit me, I changed the pool configuration a little.



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