-
Notifications
You must be signed in to change notification settings - Fork 2
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
make build work in native mode #14
base: main
Are you sure you want to change the base?
Conversation
94a0cd5
to
13c86ed
Compare
i think you might have to switch to this too: https://github.com/temporalio/sdk-java/tree/master/temporal-shaded it packages everything up including GRPC for this exact reason |
@melloware it is failing with
would you know how to solve this ? I tried to produce |
Check out this thread. A guy posted his solution in this ticket for getting it to work natively: temporalio/sdk-java#295 |
interesting its already included.. Melloware: hmmm temporal has native image stuff in their META-INF already: https://github.com/temporalio/sdk-java/blob/master/docs/AOT-native-image.md Melloware: does Quarkus ignore those META-INF GRaalVM settings? |
8112414
to
01e054b
Compare
@melloware I my startWorkers test are not working on the CI right now which is the reason why this is failing. The issue is that in order to start the workers, I need either the client to be mocked or a server to be started. I will disable both failing tests for now. I think the path forward might be to implement the dev service integration and actually start the container |
Totally agree with disabling for now. |
Native can be the very last thing we work on as its the "hardest" part. lets get everything else working. |
fix #8