Mule
  1. Mule
  2. MULE-5760

WeakHashMap in org.mule.module.scripting.expression.AbstractScriptExpressionEvaluator doesn't work

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 3.1.2
    • Fix Version/s: None
    • Component/s: Modules: Scripting
    • Labels:
      None
    • User impact:
      Medium
    • Similar Issues:
      MULE-6613HTTP outbound stream mode doesn't work
      MULE-661Spring MuleEventMulticaster doesn't work
      MULE-5476processor expression evaluator doesn't work
      MULE-275Sample errorhandler doesn't work
      MULE-1544InboundPassThroughRouter doesn't work?
      MULE-2230errorExpression on RestServiceWrapper doesn't work
      MULE-274Stop / pause of component doesn't work
      MULE-120The Axis web service provider doesn't work with Spring
      MULE-119The webap build in the latest release doesn't work
      MULE-7380Expiration doesn't work on object store
    • Effort:
      XS

      Description

      The weakHashMap in org.mule.module.scripting.expression.AbstractScriptExpressionEvaluator doesn't actually clear out keys because the keys are strings (the actual script) and the instances of org.mule.module.scripting.component.Scriptable strongly refer to the keys. See the note in http://download.oracle.com/javase/6/docs/api/java/util/WeakHashMap.html.

        Activity

        Hide
        Daniel Feist added a comment -

        Rather than fix this like this I'd prefer we either used a normal Map here or a LRU map rather than relying on a WeakHashMap.

        Show
        Daniel Feist added a comment - Rather than fix this like this I'd prefer we either used a normal Map here or a LRU map rather than relying on a WeakHashMap.

          People

          • Assignee:
            Unassigned
            Reporter:
            Craig Skinfill
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:

              Development