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

Reply via email to