KingRabbid commented on code in PR #6510:
URL: https://github.com/apache/jmeter/pull/6510#discussion_r2418210211
##########
src/core/src/main/java/org/apache/jmeter/util/JSR223TestElement.java:
##########
@@ -357,6 +355,8 @@ public void testEnded() {
*/
@Override
public void testEnded(String host) {
+ if (COMPILED_SCRIPT_CACHE.estimatedSize() > 0)
+ logger.info("Compiled cache size: {}, stats: {}",
COMPILED_SCRIPT_CACHE.estimatedSize(), COMPILED_SCRIPT_CACHE.stats());
Review Comment:
Hi,
I've patched the code; the report only is the starting point of
investigation, something which isn't available now? I'll come back in the
following days with some sample JMX to help demo the outcome.
Indeed, doesn't point to exact "missing" JSR223 element, but still it's a
starting point.
Disabling ${} is not necessary, maybe a warning is enough? Anyway, this
matters only if the variable is stored in _vars_, if defined locally in the
JSR223 then it works fine, i.e. is cached afaik.
Thanks
--
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: dev-unsubscr...@jmeter.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
