[jira] [Commented] (OFBIZ-10410) Refactor `JavaEventHandler` class
[ https://issues.apache.org/jira/browse/OFBIZ-10410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16489773#comment-16489773 ] Jacques Le Roux commented on OFBIZ-10410: - Thanks Mathieu, As I say in my commit comment: bq. jleroux: interesting, it's the 1st use in OFBiz of the "::" syntax for calling methods (cf. https://docs.oracle.com/javase/specs/jls/se8/html/jls-15.html#jls-15.13-500) I note that the string "::" is used in many places as a separator, notably in caches, as cache key separator but this should not be confusing. So you made me install Groovy Console 3.0 :) Your patch is in trunk at revision: 1832199 > 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 >Assignee: Jacques Le Roux >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)
[jira] [Commented] (OFBIZ-10410) Refactor `JavaEventHandler` class
[ https://issues.apache.org/jira/browse/OFBIZ-10410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16488733#comment-16488733 ] Mathieu Lirzin commented on OFBIZ-10410: Hello Shi, Even if I effectively worked on this while diving on OFBiz code for the REST thing, it is not related to 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)
[jira] [Commented] (OFBIZ-10410) Refactor `JavaEventHandler` class
[ https://issues.apache.org/jira/browse/OFBIZ-10410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=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)