You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@pekko.apache.org by "pjfanning (via GitHub)" <gi...@apache.org> on 2023/09/15 18:58:12 UTC

[GitHub] [incubator-pekko] pjfanning commented on a diff in pull request #650: +remote Make use of FlushConsolidationHandler.

pjfanning commented on code in PR #650:
URL: https://github.com/apache/incubator-pekko/pull/650#discussion_r1327672563


##########
remote/src/main/resources/reference.conf:
##########
@@ -568,6 +568,11 @@ pekko {
         # set to 0b for platform default
         receive-buffer-size = 256000b
 
+        # Flush operations are generally speaking expensive, consolidating flushes will improve throughput
+        # but with the downside of a bit higher latency.
+        # The number of flushes after which an explicit flush will be done, set to 0 to turn this feature off.
+        explicit-flush-after-flushes = 0

Review Comment:
   This name is confusing. What is this option for? If this value is 5, then the flush calls are ignored until we reach flush has been called 5 times?



-- 
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@pekko.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org