struts-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <>
Subject [jira] [Commented] (WW-4536) potential memory leak with annotationActionValidatorManager
Date Mon, 19 Oct 2015 18:52:28 GMT


ASF subversion and git services commented on WW-4536:

Commit b79c6959a4d5fbfd816e6dce660b095683ad987b in struts's branch refs/heads/master from
[;h=b79c695 ]

WW-4536 Checks for patterns in action names to build the same key for actions with patterns

(cherry picked from commit 30d30e07dee736243b035eb47be4a1c8dae469a2)

# Conflicts:
#	core/src/main/java/com/opensymphony/xwork2/validator/

> potential memory leak with annotationActionValidatorManager
> -----------------------------------------------------------
>                 Key: WW-4536
>                 URL:
>             Project: Struts 2
>          Issue Type: Bug
>          Components: XML Validators
>    Affects Versions: 2.3.20
>            Reporter: adam brin
>            Assignee: Lukasz Lenart
>             Fix For: 2.3.25
>         Attachments: Screen Shot 2015-08-16 at 1.48.30 PM.png, Screen Shot 2015-08-16
at 1.49.22 PM.png
> We've recently been having issues with our app crashing and trying to track down the
root causes.  One of the things we've seen is that over the course of 1 day, 50% of our memory
(almost 620 MB of 1.2 GB)  is allocated to "com.opensymphony.xwork.validator.AnnotationActionValidatorManager"
according to the heap dump. We also see millions of entries in two SynchronizedMap's:
> * validatorCache (1,635,019  entries)
> * validatorFileCache (87,321 entries)
> Looking through the entries in the validator cache, we se entries for every unique page
browsed within the app. Note: we're using staticParam mapping and NamedVariable PatternMatching
in action names e.g. {{"$\{id\}/$\{slug\}"}} so effectively every url is unique.
> Looking at the source code, should these maps perhaps be "WeakHashMaps" or WeakReferences
so that they're disposed of?
> (source code to the app is here: )

This message was sent by Atlassian JIRA

View raw message