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

[3.15 LTS] Avoid duplicated field serialization in reflection free Jackson serializers #44363

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

mariofusco
Copy link
Contributor

Backport to LTS of #44317

@quarkus-bot quarkus-bot bot changed the title [3.15 LTS] Avoid duplicated field serialization in reflection free Jackson serializers [3.15] LTS] Avoid duplicated field serialization in reflection free Jackson serializers Nov 7, 2024
@quarkus-bot quarkus-bot bot added the area/rest label Nov 7, 2024
@gsmet gsmet requested a review from rsvoboda November 7, 2024 11:50
Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

LGTM but we will discuss it in the next backport meeting so please don't merge.

@geoand geoand added triage/waiting-for-ci Ready to merge when CI successfully finishes and removed triage/waiting-for-ci Ready to merge when CI successfully finishes labels Nov 7, 2024
@geoand
Copy link
Contributor

geoand commented Nov 7, 2024

Removed the label

Copy link

quarkus-bot bot commented Nov 7, 2024

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit d694417.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.


Flaky tests - Develocity

⚙️ Native Tests - HTTP

📦 integration-tests/rest-client-reactive

io.quarkus.it.rest.client.BasicTestIT.shouldCreateClientSpans - History

  • expected: <1> but was: <2> - org.opentest4j.AssertionFailedError
org.opentest4j.AssertionFailedError: expected: <1> but was: <2>
	at io.quarkus.it.rest.client.BasicTest.shouldCreateClientSpans(BasicTest.java:216)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at io.quarkus.test.junit.QuarkusTestExtension.interceptTestMethod(QuarkusTestExtension.java:812)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)

@mariofusco mariofusco changed the title [3.15] LTS] Avoid duplicated field serialization in reflection free Jackson serializers [3.15 LTS] Avoid duplicated field serialization in reflection free Jackson serializers Nov 8, 2024
@gsmet gsmet merged commit b84b445 into quarkusio:3.15 Nov 13, 2024
52 checks passed
@gsmet gsmet added this to the 3.15.2 milestone Nov 13, 2024
@mariofusco mariofusco deleted the q44316_lts branch November 13, 2024 13:54
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.

3 participants