Hi Emily, it's great to hear there is a performance improvement (although 1% is not that much). We're working on further performance improvements in this area [1].
However, 23% additional overhead seems to me too much. Would it be possible to share your performance test case / test application? We could run similar tests in our environment and verify that there's no integration problem. Martin [1] https://issues.jboss.org/browse/WELD-1970 Dne 20.7.2015 v 11:12 Emily Jiang napsal(a): > Hi Jozef, Martin, > > Sorry for the delay! I have the performance figure collected eventually. > With this Jira's changes, it shows 1% performance improvement. This is a > step in the right direction but there is still a performance hit (23% > worse) when turning on CDI. Would you like to commit this change so that > I can reconsume this release? It will be great if a further performance > improvement can be done. I am happy to collect the performance benchmark > if needed. > > Thanks > Emily > > On Thu, Jul 2, 2015 at 8:35 AM, Jozef Hartinger <[email protected] > <mailto:[email protected]>> wrote: > > Hi Emily, > > that should not be a problem. Would it be possible for you to test > the performance improvement before we actually do the release? I can > prepare a separate branch or send you a binary. I would like to see > how much improvement this makes in your benchmark. > > Jozef > > > On 07/01/2015 06:19 PM, Emily Jiang wrote: >> Is it possible to get the following jira fix committed to >> 2.2.15.Final? >> >> https://issues.jboss.org/browse/WELD-1931 >> >> I would like to consume this and see how much improvement has been >> made. >> >> -- >> Thanks >> Emily >> ================= >> Emily Jiang >> [email protected] <mailto:[email protected]> > > > > > -- > Thanks > Emily > ================= > Emily Jiang > [email protected] <mailto:[email protected]> _______________________________________________ weld-dev mailing list [email protected] https://lists.jboss.org/mailman/listinfo/weld-dev
