Skip to content

Add XContentMediaType#6013

Closed
reta wants to merge 1 commit intoopensearch-project:mainfrom
reta:add.xcontentmediatype
Closed

Add XContentMediaType#6013
reta wants to merge 1 commit intoopensearch-project:mainfrom
reta:add.xcontentmediatype

Conversation

@reta
Copy link
Copy Markdown
Contributor

@reta reta commented Jan 25, 2023

Signed-off-by: Andriy Redko andriy.redko@aiven.io

Description

Add XContentMediaType to break XContent and MediaType coupling, plus making the change non-breaking to allow backporting to 2.x branch.

Issues Resolved

Follow up on #6009 (comment)

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
@reta
Copy link
Copy Markdown
Contributor Author

reta commented Jan 25, 2023

@nknize please

Copy link
Copy Markdown
Contributor

@nknize nknize left a comment

Choose a reason for hiding this comment

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

Let's wait on this (see comment reply). Doing this now will block refactoring other :server base classes needed to decouple the gaggs and field mapper framework.

@github-actions
Copy link
Copy Markdown
Contributor

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.repositories.azure.AzureBlobContainerRetriesTests.testWriteLargeBlob

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants