Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SOLR-15740: Convert v2 schema-edit APIs to annotations #896

Conversation

gerlowskija
Copy link
Contributor

Description

Solr's been in the slow process of moving its v2 APIs away from the
existing apispec/mapping framework towards one that relies on more
explicit annotations to specify API properties.

Solution

This commit converts all of Solr's schema-edit APIs to a single
annotation-based endpoint.

Note that the details of this conversion cause the introspect output of
this API to be lost.

Tests

See V2SchemaAPIMappingTest.

Checklist

Please review the following and check all that apply:

  • I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
  • I have created a Jira issue and added the issue ID to my pull request title.
  • I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
  • I have developed this patch against the main branch.
  • I have run ./gradlew check.
  • I have added tests for my changes.

Solr's been in the slow process of moving its v2 APIs away from the
existing apispec/mapping framework towards one that relies on more
explicit annotations to specify API properties.

This commit converts all of Solr's schema-edit APIs to a single
annotation-based endpoint.

Note that the details of this conversion cause the introspect output of
this API to be lost.
Copy link
Contributor

@epugh epugh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.... These refactors are coming fast n furious now!

@gerlowskija gerlowskija merged commit ab6d77a into apache:main Jun 8, 2022
@gerlowskija gerlowskija deleted the SOLR-15740-v2-post-schema-annotation-conversion branch June 8, 2022 10:32
gerlowskija added a commit that referenced this pull request Jun 8, 2022
Solr's been in the slow process of moving its v2 APIs away from the
existing apispec/mapping framework towards one that relies on more
explicit annotations to specify API properties.

This commit converts all of Solr's schema-edit APIs to a single
annotation-based endpoint.

Note that the details of this conversion cause the introspect output of
this API to be lost.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants