jdaugherty commented on code in PR #16272:
URL: https://github.com/apache/grails-core/pull/16272#discussion_r3916890086
##########
grails-web-url-mappings/src/main/groovy/org/grails/web/mapping/DefaultLinkGenerator.groovy:
##########
@@ -93,6 +94,9 @@ class DefaultLinkGenerator implements LinkGenerator,
PluginManagerAware {
private volatile Map<String, Set<String>> controllerNamespacesByName
private volatile GrailsClass[] cachedControllers
+ private volatile Map<String, Set<String>> controllerNamesByDomainClass
Review Comment:
We should discuss this further on the weekly or mailing list. This assumes
all applications use the same design here and it's formalizing something that
may not be true for all applications. This is also a major shift, not
necessarily a bad one, but it will require another milestone. Thus this should
go into 8.1 or later.
--
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]
