gerlowskija commented on PR #1115: URL: https://github.com/apache/solr/pull/1115#issuecomment-1307789226
Ah, I was just coming back here to close the loop but I see Houston beat me to it. There is a distinction between the APIs that include `/commands` in the path and those that don't, but Houston pointed out (correctly I think) that the distinction might be a bit subtle to expect our users to pick up on. I suspect David feels similarly. I think we should go with the majority opinion and drop `/commands` from the path here and elsewhere. > I would be willing to go through and help make the changes I'm suggesting (removing commands/ throughout the API). I wouldn't put that work on anyone else! I appreciate it, but no need. The change needed here is small and I can just make it now while I've got it pulled up. That just leaves the spreadsheet, which should be easy enough to CTRL-F on. I'd appreciate your review on the other tabs in that spreadsheet for sure though, if you've got a few @HoustonPutman. With that resolved this will be back on my list of things to test and commit. -- 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]
