patsonluk commented on PR #2672: URL: https://github.com/apache/solr/pull/2672#issuecomment-2561326911
Sorry about the delay! I was on vacation and somehow I missed this thread! To be honest I'm not totally sure why it returns null for non-select paths: ``` if (!path.endsWith("/select")) return null; ``` My guess is that the design is to avoid all non query operations on Coordinator node as those might not be supported (so to avoid any surprises). Perhaps @noblepaul could shed some light here? ✨ As for the 2 others changes, they LGTM! 😊 - adding an if to catch cases i which the collection is null - adding a node tracking property that would appear in debugQuery mode -- 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: issues-unsubscr...@solr.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org For additional commands, e-mail: issues-h...@solr.apache.org