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: [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]
