-
Notifications
You must be signed in to change notification settings - Fork 194
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
[Mandrel 23.0.0] google-bigquery integration test native compilation fails #5006
Comments
The strange thing is, that I cannot replicate the above issue. I have so far got two entirely different errors: First this:
Then that problem magically stopped happening and I now have:
Which I see is also happening on the |
I think there might be something wrong during the graal perform static analysis. The error about The |
This issue is really weird. I have run the build multiple times and got different results. Run 1: Run 2: Run 3: |
You have got a success build? Add some reflective class configurations? |
I got a successful build without adding anything new. |
Yes, I see the same as @jamesnetherton still haven't figure out what triggers it though. It looks like a data race so far... |
As discussed in oracle/graal#6904 the issue is caused by com.google.api:gax-grpc passing Ultimately this should be fixed both in GraalVM/Mandrel (see oracle/graal#6904) as well as in Till then, the fastest work around is to pass |
Thanks @zakkak!
I guess that'll be something we have to add into our Google extension docs. AFAIK, there is no way for extensions to provide arbitrary native build args (E.g via a |
That's correct, I
The main issue is that passing |
Thanks @zakkak ! I just tried with |
Not sure what dependencies are needed. I assume you are getting an https://quarkus.io/guides/native-reference#i-get-a-analysiserrorparsingerror-when-building-a-native-executable-due-to-an-unresolvedelementexception-what-can-i-do has some more info. |
Well, I still got some random results with
|
@zhfeng , this doesn't sound right. Can you please paste the output following the line
FWIW I tested with:
|
Yeah, I added The output is like
|
I just test with your command but it is still the same. |
Oh, you are right. I was testing with 999-SNAPSHOT not 3.2.0.CR1. And it turns out that with 999-SNAPSHOT I am getting an Unresolved Element exception irrespective of the value of |
Unfortunately the only work around I can think of is to introduce a native-image.properties file containing the following line:
but this file needs to be in the application, e.g. Now for an actual fix it looks like we would need a gax-grpc extension that would:
|
Oh, good catch! I just wonder why |
I can confirm that the work around works perfectly! So @jamesnetherton cam we use this work around for all google related extensions? |
OK, I found another work around to exlucde @jamesnetherton I'm not familar with |
This reverts commit 14ad98d.
@zakkak, I see that |
If you run
with each category being able to override the previous categories. So, since HTH |
Thanks a lot for the informative answer, @zakkak! |
This reverts commit 14ad98d.
…fix for PooledByteBufAllocatorL.java
…fix for PooledByteBufAllocatorL.java
Hi, |
Hi @loicmathieu , there is an upstream issue for it. See googleapis/sdk-platform-java#1344 |
Thanks, I may try to reach out to them as we already discussed the fact that adding GraalVM commands flag in |
Bug description
The initial CI fails seems to suggest there are some items that need reflective class configuration:
The text was updated successfully, but these errors were encountered: