You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I like the approach of starting big then pulling out what needs to, versus starting with too many tiny pieces. What thing I thought while looking through the refactor was maybe the metadata keys stuff could be it's own thing. I say this because it would remove some of the need for the KeyExtractorUtil to be instantiated and cached there. You could inject it instead. You couldn't do that now because it requires metadata, so it would have a circular dependency. But, if metadata was it's own thing, it would only depend on that, and the ema service would then depend on both.
Just a thought at a glance, I'm still new to the Java side of EMA.
I made it too big with #142, so let's reevaluate what is in here and move things around accordingly
The text was updated successfully, but these errors were encountered: