Skip to content

[Backport 2.x] [Refactor] MediaTypeParserRegistry to MediaTypeRegistry#9037

Merged
nknize merged 1 commit into2.xfrom
backport/backport-8940-to-2.x
Aug 1, 2023
Merged

[Backport 2.x] [Refactor] MediaTypeParserRegistry to MediaTypeRegistry#9037
nknize merged 1 commit into2.xfrom
backport/backport-8940-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

Backport 3c973ba from #8940.

This commit rote refactors MediaTypeParserRegistry to MediaTypeRegistry
to make the class naming align with the intention of the logic. The
MediaTypeRegistry is a mechanism for downstream extensions to register
concrete MediaTypes thus having Parser in the name is unneeded.

Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
(cherry picked from commit 3c973ba)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 1, 2023

Codecov Report

Merging #9037 (35c0e6e) into 2.x (35c0e6e) will not change coverage.
The diff coverage is n/a.

❗ Current head 35c0e6e differs from pull request most recent head 44f7835. Consider uploading reports for the commit 44f7835 to get more accurate results

@@            Coverage Diff            @@
##                2.x    #9037   +/-   ##
=========================================
  Coverage     70.79%   70.79%           
  Complexity    57367    57367           
=========================================
  Files          4748     4748           
  Lines        271368   271368           
  Branches      40047    40047           
=========================================
  Hits         192110   192110           
  Misses        62775    62775           
  Partials      16483    16483           

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Aug 1, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Aug 1, 2023

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.search.SearchWeightedRoutingIT.testStrictWeightedRoutingWithCustomString
      1 org.opensearch.remotestore.multipart.RemoteStoreMultipartIT.testRemoteTranslogRestoreWithNoDataPostRefresh

@nknize nknize merged commit f25e217 into 2.x Aug 1, 2023
@nknize nknize deleted the backport/backport-8940-to-2.x branch August 1, 2023 20:49
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