Page MenuHomePhabricator

search/glent fails on Java 11
Open, Needs TriagePublic3 Estimated Story Points

Description

As part of T350587 I have added Java 11 in CI for search/glent and that fails with a dummy change https://gerrit.wikimedia.org/r/c/search/glent/+/973357

[ERROR] Failures: 
[ERROR]   NormalizersTest.testSimpleLowerCase:34->testNormalizer:61 expected:<киану..ꮳꮃꭹ,?[?𞤁𞤂𞤀𞤃]-πάθος. džijⅲ. daß 𝐀?...> but was:<киану..ꮳꮃꭹ,?[?𞤣𞤤𞤢𞤥]-πάθος. džijⅲ. daß 𝐀?...>
[INFO] 
[ERROR] Tests run: 70, Failures: 1, Errors: 0, Skipped: 0

The Java 8 based job passes though :)

Event Timeline

@hashar just realized that glent being run by spark we might still want it to be built with jdk8.
The failure with jdk11 is unexpected tho, so definitely something we should look into, thanks!

Gehel set the point value for this task to 3.Nov 13 2023, 4:46 PM

Not sure what to do about Spark, but the Java 11 failure is arguably a feature, not a bug! The script that changed is Adlam, and Java 11 got smarter about it. Some of the other changes lsted there make me wonder what other texty corner cases are going to be affected by the upgrade.

dr0ptp4kt subscribed.

Closing this out until newer Java comes to the analytics cluster.

Change #1031958 had a related patch set uploaded (by Gehel; author: Gehel):

[search/glent@master] test: Disable testSimpleLowerCase() on JRE 11.

https://gerrit.wikimedia.org/r/1031958

Change #1031958 merged by jenkins-bot:

[search/glent@master] test: Disable testSimpleLowerCase() on JRE 11.

https://gerrit.wikimedia.org/r/1031958