[GitHub] [solr] gerlowskija commented on pull request #1358: SOLR-16488: Convert ZookeeperReadAPI to JAX-RS

["gerlowskija (via GitHub)" <gi...@apache.org>]

gerlowskija commented on PR #1358:
URL: https://github.com/apache/solr/pull/1358#issuecomment-1430406466

   FYI @joshgog - I think this PR should serve as a successor to #1144, in light of my initial mistake overlooking these v2 APIs.
   
   This PR leaves the endpoints themselves unchanged (in terms of HTTP verb, path, etc.).  But I do think, while we're touching these APIs and since we're going through this process for all of our v2 APIs, that we should rethink what these APIs look like.
   
   (Most of the API-appearance changes have already been proposed, reviewed, etc. in bulk, but obviously these APIs missed that since I didn't find them until just recently.)
   
   I'll post suggestions for those API changes shortly.


-- 
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-unsubscribe@solr.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@solr.apache.org
For additional commands, e-mail: issues-help@solr.apache.org