[jira] [Commented] (OFBIZ-10410) Refactor `JavaEventHandler` class

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (OFBIZ-10410) Refactor `JavaEventHandler` class

Nicolas Malin (Jira)

    [ https://issues.apache.org/jira/browse/OFBIZ-10410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16488619#comment-16488619 ]

Shi Jinghai commented on OFBIZ-10410:
-------------------------------------

Thank you Mathieu!

I'm not sure whether the get/post method attribute of request-map is included in this issue scope, I saw you mentioned it in your comment in OFBIZ-4274.

> Refactor `JavaEventHandler` class
> ---------------------------------
>
>                 Key: OFBIZ-10410
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10410
>             Project: OFBiz
>          Issue Type: Improvement
>    Affects Versions: Trunk
>            Reporter: Mathieu Lirzin
>            Priority: Minor
>         Attachments: OFBIZ-10410-refactor-java-event-handler.patch
>
>
> Instead of relying on manual synchronisation, use a concurrent map for caching loaded classes.  Inline private `invoke` delegate which intent was fuzzy.  Spread arguments when calling `Method::invoke` and `Class::getMethod`.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)