Replies: 2 comments
-
|
That sounds great! I’ve also noticed that AttributeDataAccessor shows up in profiles, so focusing there makes sense. Replacing reflection with mapper-generated accessors and reusing existing property mappings sounds like a solid direction. |
Beta Was this translation helpful? Give feedback.
-
|
I thought of creating the mappings by hand. (Or should mapperly depend on itself?) |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
Hi @latonz,
I'm considering doing some performance improvement on the generation. Are you interested in it?
What my gut feeling, and some performance tests showed, is that the
AttributeDataAccessorhas the biggest impact on the performance. So I thought I would improve that.If you already have some idea about improvements, I would like to hear it.
My main idea is to replace the reflections with mapper methods, re-using the property mappings that are there.
Also, I would add caching on this level too, so the complicated processing doesn't need to be repeated. WDYT?
Beta Was this translation helpful? Give feedback.
All reactions