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

Upgrade to Hibernate Search 6.0.3.Final #16467

Merged
merged 3 commits into from
Apr 14, 2021
Merged

Upgrade to Hibernate Search 6.0.3.Final #16467

merged 3 commits into from
Apr 14, 2021

Conversation

yrodiere
Copy link
Member

@yrodiere yrodiere commented Apr 13, 2021

Relates to #16463, and provides a workaround for Hibernate Search, but does not fix the problem entirely: we would need a fix in ORM for that.

CC @gsmet

Creating as draft: we need a release of Hibernate Search first; one that includes hibernate/hibernate-search#2544 (HSEARCH-4211) . I tested that the upgrade fixes the problem locally.

@quarkus-bot quarkus-bot bot added area/dependencies Pull requests that update a dependency file area/hibernate-search Hibernate Search labels Apr 13, 2021
@yrodiere yrodiere marked this pull request as ready for review April 13, 2021 15:49
@yrodiere
Copy link
Member Author

@gsmet Ready for review; I released Hibernate Search 6.0.3.Final and it should be available in Maven Central soon.

@yrodiere yrodiere requested a review from gsmet April 13, 2021 15:50
@gsmet
Copy link
Member

gsmet commented Apr 13, 2021

Let's wait for the sync to Central.

@quarkus-bot
Copy link

quarkus-bot bot commented Apr 13, 2021

This workflow status is outdated as a new workflow run has been triggered.

Failing Jobs - Building 9ed7581

⚠️ Artifacts of the workflow run were not available thus the report misses some details.

Status Name Step Test failures Logs Raw logs
Initial JDK 11 Build ⚠️ Check → Logs Raw logs
Attach pull request number ⚠️ Check → Logs Raw logs
CI Sanity Check ⚠️ Check → Logs Raw logs
🚫 Devtools Tests - JDK ${{ matrix.java.name }}
🚫 Devtools Tests - JDK 11 Windows
🚫 Gradle Tests - JDK 11 ${{ matrix.os.family }}
🚫 JVM Tests - JDK ${{ matrix.java.name }}
🚫 JVM Tests - JDK 11 Windows
🚫 Maven Tests - JDK ${{ matrix.java.name }}
🚫 Maven Tests - JDK 11 Windows
🚫 MicroProfile TCKs Tests
🚫 Native Tests - ${{ matrix.category }}
Native Tests - Read JSON matrix ⚠️ Check → Logs Raw logs
🚫 Native Tests - Windows - ${{ matrix.category }}

It causes weird errors when mass indexing, probably because
the MethodHandles implementation in native mode is not completely
thread-safe.

For example:

Entities that could not be indexed correctly:
Person#212 : org.hibernate.search.util.common.SearchException:
HSEARCH900005: Exception while invoking 'private java.lang.Long
io.quarkus.it.hibernate.search.elasticsearch.search.Person.id' on
'io.quarkus.it.hibernate.search.elasticsearch.search.Person@33be79fb'.
...
Caused by: java.lang.ClassCastException:
io.quarkus.it.hibernate.search.elasticsearch.search.Address cannot be
cast to io.quarkus.it.hibernate.search.elasticsearch.search.Person
@gsmet gsmet merged commit 2cb3c8a into quarkusio:main Apr 14, 2021
@quarkus-bot quarkus-bot bot added this to the 2.0 - main milestone Apr 14, 2021
@gsmet gsmet modified the milestones: 2.0 - main, 1.13.2.Final Apr 14, 2021
@yrodiere yrodiere deleted the i16463 branch May 31, 2022 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependencies Pull requests that update a dependency file area/hibernate-search Hibernate Search
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants