gerlowskija commented on PR #2144: URL: https://github.com/apache/solr/pull/2144#issuecomment-1909003780
@iamsanjay - I would love to see this make it into 9.5, so I took the liberty of making a few of the changes that were outstanding from above. Namely: - Changing the API to be `/api/node/commands/<someId>`, to avoid the unused-coreName problem. - Restructuring your unit tests to avoid use of JerseyTest. As we discussed this already above I don't expect you'll object to any of this, but if you do lmk and I'm happy to revert my recent commits on the branch while we discuss. But otherwise, my next steps are to update CHANGES.txt, and get the tests passing so we can merge this soon! -- 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
