Skip to content

Conversation

mbellade
Copy link
Member

@mbellade mbellade commented Aug 1, 2025

Alternative check for JSON/XML properties without iterating through the entire metamodel's properties/columns.

cc @marko-bekhta

@mbellade mbellade added the area/hibernate-orm Hibernate ORM label Aug 1, 2025
Copy link

quarkus-bot bot commented Aug 1, 2025

/cc @gsmet (hibernate-orm)

This comment has been minimized.

This comment has been minimized.

@mbellade
Copy link
Member Author

mbellade commented Aug 1, 2025

Added a second commit which handles JSON_ARRAY / XML_ARRAY, which need to be checked slightly differently. This will require https://hibernate.atlassian.net/browse/HHH-19675 to be fixed though, so we'll need to wait for that.

Copy link

quarkus-bot bot commented Aug 1, 2025

Status for workflow Quarkus CI

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

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

@gsmet
Copy link
Member

gsmet commented Aug 1, 2025

Closing in favor of #49275 that also adds tests.

I will merge #49275 once the Hibernate ORM upgrade is merged in the tree.

@gsmet gsmet closed this Aug 1, 2025
@quarkus-bot quarkus-bot bot added the triage/invalid This doesn't seem right label Aug 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/hibernate-orm Hibernate ORM triage/invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants