jdaugherty commented on PR #16130: URL: https://github.com/apache/grails-core/pull/16130#issuecomment-5360441956
@codeconsole I think we need to look at the histoyro f the changes on the navigablemap. from our call yesterday: 2016 – #10188: buggy/awkward nested-map replacement semantics are identified; Jeff investigates and adds a failing test. 2016 – #10280: Jeff starts an experimental implementation to fix the semantics. It never gets merged. 2020 – #11554: NavigableMap is deprecated, making Jeff's unfinished approach ostensibly unnecessary. 2024 – #13869: Grails reverses course enough to restore dot-notation navigation for Grails 7. 2025 – #14005: the project explicitly resurrects #10188/#10280 as the underlying problems that should now actually be fixed. I don't think memoized is the solution here b/c it's a cache and with large configuration you're going to be using a lot of memory. Can't we look at what Jeff did originally and see if we can fix it that way? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
