https://pinot.apache.org/ logo
j

jiatao

03/16/2021, 8:52 PM
Hi,
Pinot Quickstart on JDK 15-ea
is failing for my pr (which only change log messages). Seems like the test is running java 16 instead of 15:
JAVA_HOME_16.0.0_x64=/opt/hostedtoolcache/jdk/16.0.0/x64
. Any idea how to fix this? The pr test link for reference: https://github.com/apache/incubator-pinot/pull/6684/checks?check_run_id=2124725931
x

Xiang Fu

03/16/2021, 9:21 PM
Rerun the test doesn’t help?
j

jiatao

03/16/2021, 9:24 PM
I changed one line, and rebase the pr which trigger the test again, but it's still running jdk 16.
x

Xiang Fu

03/16/2021, 9:44 PM
I saw it’s failing for other PR as well
guess the issue is on github action side
j

jiatao

03/16/2021, 9:59 PM
I see. Thanks.
FYI: ^^ @Jack
j

Jack

03/16/2021, 10:01 PM
sorry I have no context on this issue in apache pinot repo though..
One thing worthy to try is to use java 15 GA version instead of EA: https://github.com/apache/incubator-pinot/pull/6687
Build passes. @Xiang Fu@jiatao we should be good to go with that changes now 👆
x

Xiang Fu

03/16/2021, 10:49 PM
👍
j

jiatao

03/16/2021, 10:50 PM
@Jack Thanks!
c

Chad Preisler

03/17/2021, 2:58 AM
Is there a special branch for building this on JDK 15? I couldn’t get it to build
j

jiatao

03/17/2021, 3:00 AM
The fix from Jack has been merged to master. I rebased my pr and it's building with JDK 15 now.
2 Views