kusalk commented on code in PR #766:
URL: https://github.com/apache/struts/pull/766#discussion_r1359879865


##########
core/src/main/resources/org/apache/struts2/default.properties:
##########
@@ -243,25 +243,25 @@ struts.ognl.enableExpressionCache=true
 ### For expressionCacheLRUMode true, the limit will ensure the cache does not 
exceed
 ### that size, dropping the oldest (least-recently-used) expressions to add 
new ones.
 ### NOTE: If not set, the default is 25000, which may be excessive.
-# struts.ognl.expressionCacheMaxSize=1000
+struts.ognl.expressionCacheMaxSize=10000

Review Comment:
   I ended up defining the defaults in the properties file and removing 
defaults from the code - except for required values when bootstrapping the 
container. (And except for a deprecated constructor which is awaiting deletion)



-- 
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: issues-unsubscr...@struts.apache.org

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

Reply via email to