dsmiley commented on PR #3305: URL: https://github.com/apache/solr/pull/3305#issuecomment-3199057665
I addressed the pressing matters. This is ready to merge, and is back-portable to 9.10. I plan to do so Thursday. The properties include the name of the node. I'm not sure why this was added, as it's redundant with the final path component name to the live node. Should I remove it? CC @HoustonPutman Admittedly there's no test to read the role data. I have no current plans to read it; it's nice-to-have. Some caching would be nice but at least the use case I have in mind, it wouldn't be necessary. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
