Skip to content

Commit

Permalink
fix(java): restore native image configurations (#1844)
Browse files Browse the repository at this point in the history
This configuration was added in https://togithub.com/googleapis/java-bigquerystorage/pull/1488 but was removed by owlbot in https://togithub.com/googleapis/java-bigquerystorage/pull/1543. The build script changes were excluded from https://togithub.com/googleapis/java-bigquerystorage/pull/1362 so this missing configuration was only caught through manual testing by running `mvn test -Pnative`. This PR restores both changes.
  • Loading branch information
mpeddada1 committed Oct 25, 2022
1 parent 0d396f4 commit 8ce670a
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 0 deletions.
1 change: 1 addition & 0 deletions .github/.OwlBot.yaml
Expand Up @@ -83,6 +83,7 @@ deep-preserve-regex:
- "/google-cloud-bigquerystorage/src/main/java/com/google/cloud/bigquery/storage/v1/Exceptions.java"
- "/google-cloud-bigquerystorage/src/main/java/com/google/cloud/bigquery/storage/v1/ConnectionWorker.java"
- "/google-cloud-bigquerystorage/src/main/java/com/google/cloud/bigquery/storage/v1/ConnectionWorkerPool.java"
- "/google-cloud-bigquerystorage/src/main/resources/META-INF/native-image/"

deep-copy-regex:
- source: "/google/cloud/bigquery/storage/(v.*)/.*-java/proto-google-.*/src"
Expand Down
10 changes: 10 additions & 0 deletions .kokoro/build.sh
Expand Up @@ -68,6 +68,16 @@ integration)
verify
RETURN_CODE=$?
;;
graalvm)
# Run Unit and Integration Tests with Native Image
mvn -B ${INTEGRATION_TEST_ARGS} -ntp -Pnative -Penable-integration-tests test
RETURN_CODE=$?
;;
graalvm17)
# Run Unit and Integration Tests with Native Image
mvn -B ${INTEGRATION_TEST_ARGS} -ntp -Pnative -Penable-integration-tests test
RETURN_CODE=$?
;;
samples)
SAMPLES_DIR=samples
# only run ITs in snapshot/ on presubmit PRs. run ITs in all 3 samples/ subdirectories otherwise.
Expand Down
@@ -0,0 +1,7 @@
[
{
"name":"java.lang.Object",
"allDeclaredFields":true,
"queryAllDeclaredMethods":true,
"methods":[{"name":"<init>","parameterTypes":[] }]}
]

0 comments on commit 8ce670a

Please sign in to comment.