KEMBAR78
test: fix graalvm native test B by diegomarquezp · Pull Request #3775 · googleapis/java-bigquery · GitHub
Skip to content

Conversation

@diegomarquezp
Copy link
Contributor

@diegomarquezp diegomarquezp commented May 5, 2025

Fixes #3760

BEGIN_COMMIT_OVERRIDE
test: fix graalvm native test B
END_COMMIT_OVERRRIDE

This includes gax-testlib with the necessary build-time initialization flags (see properties file).

Additionally, presubmit jobs for GraalVM A, B and C were added.

@diegomarquezp diegomarquezp added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label May 5, 2025
@product-auto-label product-auto-label bot added size: s Pull request size is small. api: bigquery Issues related to the googleapis/java-bigquery API. labels May 5, 2025
@diegomarquezp diegomarquezp removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label May 5, 2025
@diegomarquezp diegomarquezp marked this pull request as ready for review May 5, 2025 17:31
@diegomarquezp diegomarquezp requested review from a team as code owners May 5, 2025 17:31
@diegomarquezp diegomarquezp requested a review from alvarowolfx May 5, 2025 17:31
@product-auto-label product-auto-label bot added size: m Pull request size is medium. and removed size: s Pull request size is small. labels May 5, 2025
@diegomarquezp
Copy link
Contributor Author

@PhongChuong the jobs were successfully rolled out as presubmits.

image

@diegomarquezp diegomarquezp requested a review from PhongChuong May 6, 2025 14:29
@diegomarquezp diegomarquezp merged commit 782aa77 into main May 6, 2025
22 checks passed
@diegomarquezp diegomarquezp deleted the fix-graal-b branch May 6, 2025 14:59
@diegomarquezp diegomarquezp changed the title fix: fix graalvm native test B test: fix graalvm native test B May 6, 2025
lqiu96 pushed a commit that referenced this pull request May 7, 2025
* fix: fix graalvm native test B

* fix: add gax testlib to java-bigquery

This includes the necessary build-time initialization flags (see [properties file](https://github.com/googleapis/sdk-platform-java/blob/1eb107d1c09d8a2182fe0f550bf23e3ca3ac414f/gax-java/gax/src/test/resources/META-INF/native-image/com.google.api/gax/native-image.properties)

* chore: add gax testlib to dependency analyzer exemptions

* chore: fix gax package reference

* chore: convert tabs to spaces

* chore: add presubmit checks for graalvm

* chore: restore integration job after testing

* chore: restore return code in integration test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigquery Issues related to the googleapis/java-bigquery API. size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bigquery.it.ITRemoteUDFTest: testRoutineRemoteUDF failed

2 participants