ningyougang commented on a change in pull request #5102:
URL: https://github.com/apache/openwhisk/pull/5102#discussion_r627863269



##########
File path: core/invoker/src/main/resources/application.conf
##########
@@ -60,9 +60,13 @@ whisk {
     user-memory: 1024 m
     concurrent-peek-factor: 0.5 #factor used to limit message peeking: 0 < 
factor <= 1.0 - larger number improves concurrent processing, but increases 
risk of message loss during invoker crash
     akka-client:  false # if true, use PoolingContainerClient for HTTP from 
invoker to action container (otherwise use ApacheBlockingContainerClient)
-    prewarm-expiration-check-interval: 1 minute # period to check for prewarm 
expiration
+    prewarm-expiration-check-init-delay: 10 minute # the init delay time for 
the first check
+    prewarm-expiration-check-interval: 10 minute # period to check for prewarm 
expiration
     prewarm-expiration-check-interval-variance: 10 seconds # varies expiration 
across invokers to avoid many concurrent expirations
     prewarm-expiration-limit: 100 # number of prewarms to expire in one 
expiration cycle (remaining expired will be considered for expiration in next 
cycle)
+    prewarm-max-retry-limit: 5 # max retry limit for create prewarm
+    prewarm-promotion: false # if true, action can take prewarm container 
which has bigger memory

Review comment:
       Yes, but for `prewarm-promotion`, is used in 
FunctionPullingContainerPool only, and the configuration is `false` which mean, 
didn't take prewarm container which has bigger memory




-- 
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:
us...@infra.apache.org


Reply via email to