-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
GH-40619: [Java] JDBC Adapter Build Issue #40656
Conversation
@github-actions crossbow submit verify-rc-source-java-linux-almalinux-8-amd64 |
@github-actions crossbow submit java |
@github-actions crossbow submit verify-rc-source-java |
|
Revision: a71a75e Submitted crossbow builds: ursacomputing/crossbow @ actions-da05b0cde8
|
Revision: a71a75e Submitted crossbow builds: ursacomputing/crossbow @ actions-0be4c494fc |
@github-actions crossbow submit java |
Revision: a71a75e Submitted crossbow builds: ursacomputing/crossbow @ actions-e0930787da |
It seems like this PR is not breaking any CIs. |
Probably better to wait until the issues which are unrelated to this modification though validates it. |
@github-actions crossbow submit -g java |
Revision: 27ec5dc Submitted crossbow builds: ursacomputing/crossbow @ actions-80ff4b4599 |
|
After merging your PR, Conbench analyzed the 6 benchmarking runs that have been run so far on merge-commit b8e5b51. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. It also includes information about 2 possible false positives for unstable benchmarks that are known to sometimes produce them. |
Rationale for this change
Compiling JDBC Adapter module fails when running from a time zone which is not UTC. The tests assume the set time zone is UTC. Since running from a location where the timezone is not UTC, the error documented in #40619 occurs.
What changes are included in this PR?
Adding the time zone as a property to maven profile.
Are these changes tested?
These changes have been tested locally in a non-UTC timezone (+0530).
Are there any user-facing changes?
No