[GitHub] [solr] gerlowskija commented on pull request #1453: SOLR-16391: Migrate coll-creation API to JAX-RS

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

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

   As this PR stands today, the new API and refactoring appear to work correctly for v1 requests, but actually fails for v2 requests.
   
   The root of the problem is that multiple APIs now claim `POST /api/collections` as their endpoint.  (See the v2 backup, restore, create alias, set-alias-property, and delete alias APIs).
   
   This wasn't a problem while the APIs all shared the legacy framework, as the legacy framework is designed to inspect a "command" indicator at the top level of the request body (e.g. `{"delete-alias": {...}}).  But moving "create" over to JAX-RS while the other APIs remain on legacy proved to be a problem: V2HttpCall sends incoming "create" requests to the legacy framework, which no longer handles the "create" command, so an error is thrown.
   
   This isn't a problem long-term, but we won't be able to proceed here until the other `POST /collections` APIs are moved over to JAX-RS.


-- 
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