You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@ignite.apache.org by GitBox <gi...@apache.org> on 2021/01/29 11:37:29 UTC

[GitHub] [ignite] isapego commented on a change in pull request #8726: IGNITE-13763: Add connection limit for C++ thin client

isapego commented on a change in pull request #8726:
URL: https://github.com/apache/ignite/pull/8726#discussion_r566761916



##########
File path: modules/platforms/cpp/thin-client/include/ignite/thin/ignite_client_configuration.h
##########
@@ -230,6 +231,29 @@ namespace ignite
                 return partitionAwareness;
             }
 
+            /**
+             * Get connection limit.
+             * Zero means that number of active connections is not limited.

Review comment:
       But it is unsigned. If you set -1 compiler will give you either a warning or error depending on your compiler flags.




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

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