gerlowskija opened a new pull request, #1053:
URL: https://github.com/apache/solr/pull/1053

   # Description
   
   The Solr community is engaged in an ongoing effort to make Solr's v2 API 
more REST-ful and intuitive for users so that it can eventually serve as a 
viable v1 replacement. This PR in particular focuses on Solr's 
"delete-replica-prop" API, aligning it with the API design described 
[here](https://docs.google.com/spreadsheets/d/1HAoBBFPpSiT8mJmgNZKkZAPwfCfPvlc08m5jz3fQBpA/edit?usp=sharing).
   
   In particular this changes the v2 API from:
   
   ```
   POST /collections/collName {"delete-replica-property" : {...}}
   ```
   
   to:
   
   ```
   DELETE 
/collections/<coll>/shards/<shard>/replicas/<replica>/properties/<prop>
   ```
   
   This is a breaking change for users of the v2 API, but one that is allowed 
due to v2's "experimental" designation.
   
   # Solution
   
   This PR modifies delete-replica-prop in the way described, but it also 
refactors the API class to use the newly added JAX-RS framework in the process. 
This framework-switchover, while conceptually separate from the API realignment 
in this case, makes sense as both tasks were coming in the short term and touch 
identical chunks of code.
   
   As a result, this PR not only changes the cosmetics of the 
delete-replica-prop endpoint, it also adds it to Solr's growing OpenAPI spec.
   
   # Tests
   
   See DeleteReplicaPropertyAPITest for new unit tests.
   
   # Checklist
   
   Please review the following and check all that apply:
   
   - [x] I have reviewed the guidelines for [How to 
Contribute](https://wiki.apache.org/solr/HowToContribute) and my code conforms 
to the standards described there to the best of my ability.
   - [x] I have created a Jira issue and added the issue ID to my pull request 
title.
   - [x] I have given Solr maintainers 
[access](https://help.github.com/en/articles/allowing-changes-to-a-pull-request-branch-created-from-a-fork)
 to contribute to my PR branch. (optional but recommended)
   - [x] I have developed this patch against the `main` branch.
   - [ ] I have run `./gradlew check`.
   - [x] I have added tests for my changes.
   - [ ] I have added documentation for the [Reference 
Guide](https://github.com/apache/solr/tree/main/solr/solr-ref-guide)
   


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

Reply via email to