-
Notifications
You must be signed in to change notification settings - Fork 14
pyhdk from conda-forge segfaults #51
Comments
The log attached |
It looks like it could be a java 11 error. I am not seeing the issue with openjdk 8. I have some other questions while I continue to investigate:
|
there were some issues with 8 during conda packaging @vlad-penkin can comment more. I will try 8
right, we should fix this
Yeah, it's not necessary, it should run without it |
Looks like this is the stack trace:
|
openjdk 8 is almost EOL and not well maintained from the security perspective. openjdk 11 in preffered, conda-forge recipe pins 11. |
Reproduces on openjdk 11, but not on 17 ( Looks like this might be an issue with 11: Can we go to 17? |
Yes, it seems it works, testing conda-forge/hdk-feedstock#3 |
Here is a small update on non-conda build. It does not work with openjdk-17-jdk. Our docker builds use docker/dev/Dockerfile.* docker files which download cmake from github. The file cmake-3.24.1/Modules/FindJNI.cmake does not have an ubuntu path for openjdk-17-jdk, which results in "cannot find JNI" cmake error. How this works in a conda env I don't know. Maybe FindJNI falls back to a system JDK.
|
Which container does that use? Looks like we use openjdk 11 here: https://github.com/intel-ai/hdk/blob/main/docker/Dockerfile |
I’be tried to change to openjdk-17-jdk and then switched it back https://github.com/intel-ai/omniscidb/runs/8128359640?check_suite_focus=true |
I see here https://github.com/intel-ai/omniscidb/runs/8105954470?check_suite_focus=true that openjdk-17 works fine and there is |
@Garra1980 added CmakeCache.txt
|
I've patched cmake ( |
It seems maven cannot work with openjdk-17
|
Here the related case is described https://stackoverflow.com/questions/44438848/maven-crashes-when-trying-to-compile-a-project-error-executing-maven |
This is weird, likely means that installation is not correct |
"This is weird, likely means that installation is not correct" IMHO the problem is maven build is linked to jdk, the same problem is descibed on Stackoverflow, see the link above |
Everything works with conda so should do without it this is my point |
Closing as fixed, issues with docker will be handled in separate issue |
Here is the scenario:
Here is the error:
The text was updated successfully, but these errors were encountered: