...
[15:44:32 CST(-0600)] <jwennmacher> If using compile-time or run-time weaving, this type of issue of course wouldn't occur.
[15:45:11 CST(-0600)] <EricDalquist> not sure I follow
[15:45:25 CST(-0600)] <EricDalquist> JpaAggregatedTabLookupDao.getTabMapping has the @OpenEntityManager annotation
[15:45:40 CST(-0600)] <EricDalquist> ah right
[15:45:41 CST(-0600)] <EricDalquist> I see
[15:45:47 CST(-0600)] <EricDalquist> ignore my previous statement
[15:46:08 CST(-0600)] <EricDalquist> yeah, I have nothing against compile-time weaving if someone (hint) wants to look into it
[15:46:14 CST(-0600)] <jwennmacher> Yes. The controller was invoking getMappedTabForLayoutId (incorreclty mind you ... I thought the layoutNodeId was the id, but I found out it wasn't and had to implement a get by id method).
[15:46:23 CST(-0600)] <EricDalquist> load-time worries me a little more since then we're adding more complexity to the deployers
[15:46:27 CST(-0600)] <jwennmacher> I like that hint.
[15:46:59 CST(-0600)] <jwennmacher> Yeah I like compile-time, especially on big projects. You don't need anything else happening at startup.
[15:47:10 CST(-0600)] <jwennmacher> Slows things down even more
[15:48:07 CST(-0600)] <jwennmacher> I didn't know if there was discussion about that before or not. I've used it before and I generally find it better than aop proxies because you don't have to think about these types of things.
[15:48:46 CST(-0600)] <EricDalquist> no, its never really been brought up before
[15:48:53 CST(-0600)] <EricDalquist> I've thought abuot it
[15:49:01 CST(-0600)] <EricDalquist> but just never had the motivation to look into it
[15:49:08 CST(-0600)] <EricDalquist> I'd love to see something like that figured out though
[15:49:24 CST(-0600)] <EricDalquist> I suppose in theory runtime performance would be better with compile-time weaving too
[15:49:30 CST(-0600)] <EricDalquist> fewer wrapper layers
[15:49:44 CST(-0600)] <jwennmacher> If there are any downsides you can think of, I'm interested in hearing that. Otherwise, I'll add that as a todo item. I suspect it might be considered low on the priority list though (which is fine)
[15:50:02 CST(-0600)] <EricDalquist> yeah, I don't know enough about it to really know of down sides
[15:50:07 CST(-0600)] <EricDalquist> but I can't think of any right now
[15:50:16 CST(-0600)] <EricDalquist> just ping here when/if you work on it
[15:50:21 CST(-0600)] <EricDalquist> I'd love to see the possibilities
[15:50:31 CST(-0600)] <jwennmacher> Sounds good. Always love to get others insights.