Skip to content

fix: improve 503 handling for json resumable uploads#2987

Merged
BenWhitehead merged 1 commit into
googleapis:mainfrom
benjaminp:json-npe
Mar 14, 2025
Merged

fix: improve 503 handling for json resumable uploads#2987
BenWhitehead merged 1 commit into
googleapis:mainfrom
benjaminp:json-npe

Conversation

@benjaminp

Copy link
Copy Markdown
Contributor

Port NullPointerException fix of JsonResumableSessionPutTask in 9b4bb82 to JsonResumableSessionQueryTask, which has similar code.

Port `NullPointerException` fix of `JsonResumableSessionPutTask` in 9b4bb82 to `JsonResumableSessionQueryTask`, which has similar code.
@benjaminp benjaminp requested a review from a team March 14, 2025 06:18
@product-auto-label product-auto-label Bot added size: s Pull request size is small. api: storage Issues related to the googleapis/java-storage API. labels Mar 14, 2025
@BenWhitehead BenWhitehead added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Mar 14, 2025
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Mar 14, 2025
@BenWhitehead BenWhitehead merged commit 9bc2b14 into googleapis:main Mar 14, 2025
@benjaminp benjaminp deleted the json-npe branch March 14, 2025 20:08
MartijnVisser added a commit to MartijnVisser/flink that referenced this pull request Jun 1, 2026
…nk-gs-fs-hadoop

The GCS file system bundled google-cloud-storage 2.29.1, which throws a
NullPointerException instead of retrying certain GCS 503 Service Unavailable
errors during resumable uploads, breaking checkpointing for jobs writing to
gs:// via a RecoverableWriter. The upstream fix is in googleapis/java-storage#2987.

Bump google-cloud-storage 2.29.1 -> 2.68.0 and the matching grpc artifacts
1.59.1 -> 1.81.0, regenerate the bundled-dependency NOTICE accordingly, add the
bundled license file for the newly bundled stax2-api, and update the version
links in the GCS filesystem documentation.

The newly pulled-in jspecify dependency provides only static-analysis nullness
annotations that are not needed at runtime, so it is excluded from the shaded
jar like the other annotation-only dependencies (checker-qual,
error_prone_annotations, j2objc-annotations).

Generated-by: Claude Code (Opus 4.8)
MartijnVisser added a commit to apache/flink that referenced this pull request Jun 3, 2026
…nk-gs-fs-hadoop (#28286)

* [FLINK-39136][filesystems] Bump google-cloud-storage to 2.68.0 in flink-gs-fs-hadoop

The GCS file system bundled google-cloud-storage 2.29.1, which throws a
NullPointerException instead of retrying certain GCS 503 Service Unavailable
errors during resumable uploads, breaking checkpointing for jobs writing to
gs:// via a RecoverableWriter. The upstream fix is in googleapis/java-storage#2987.

Bump google-cloud-storage 2.29.1 -> 2.68.0 and the matching grpc artifacts
1.59.1 -> 1.81.0, regenerate the bundled-dependency NOTICE accordingly, add the
bundled license file for the newly bundled stax2-api, and update the version
links in the GCS filesystem documentation.

Add integration tests that run against a real GCS bucket, mirroring the existing
S3 filesystem integration tests. They are skipped unless a bucket is configured
via the IT_CASE_GCS_BUCKET environment variable; authentication uses Application
Default Credentials (GOOGLE_APPLICATION_CREDENTIALS).

Generated-by: Claude Code (Opus 4.8)
MartijnVisser added a commit to apache/flink that referenced this pull request Jun 3, 2026
…nk-gs-fs-hadoop (#28286)

* [FLINK-39136][filesystems] Bump google-cloud-storage to 2.68.0 in flink-gs-fs-hadoop

The GCS file system bundled google-cloud-storage 2.29.1, which throws a
NullPointerException instead of retrying certain GCS 503 Service Unavailable
errors during resumable uploads, breaking checkpointing for jobs writing to
gs:// via a RecoverableWriter. The upstream fix is in googleapis/java-storage#2987.

Bump google-cloud-storage 2.29.1 -> 2.68.0 and the matching grpc artifacts
1.59.1 -> 1.81.0, regenerate the bundled-dependency NOTICE accordingly, add the
bundled license file for the newly bundled stax2-api, and update the version
links in the GCS filesystem documentation.

Add integration tests that run against a real GCS bucket, mirroring the existing
S3 filesystem integration tests. They are skipped unless a bucket is configured
via the IT_CASE_GCS_BUCKET environment variable; authentication uses Application
Default Credentials (GOOGLE_APPLICATION_CREDENTIALS).

Generated-by: Claude Code (Opus 4.8)
MartijnVisser added a commit to apache/flink that referenced this pull request Jun 3, 2026
…nk-gs-fs-hadoop (#28286)

* [FLINK-39136][filesystems] Bump google-cloud-storage to 2.68.0 in flink-gs-fs-hadoop

The GCS file system bundled google-cloud-storage 2.29.1, which throws a
NullPointerException instead of retrying certain GCS 503 Service Unavailable
errors during resumable uploads, breaking checkpointing for jobs writing to
gs:// via a RecoverableWriter. The upstream fix is in googleapis/java-storage#2987.

Bump google-cloud-storage 2.29.1 -> 2.68.0 and the matching grpc artifacts
1.59.1 -> 1.81.0, regenerate the bundled-dependency NOTICE accordingly, add the
bundled license file for the newly bundled stax2-api, and update the version
links in the GCS filesystem documentation.

Add integration tests that run against a real GCS bucket, mirroring the existing
S3 filesystem integration tests. They are skipped unless a bucket is configured
via the IT_CASE_GCS_BUCKET environment variable; authentication uses Application
Default Credentials (GOOGLE_APPLICATION_CREDENTIALS).

Generated-by: Claude Code (Opus 4.8)
MartijnVisser added a commit to MartijnVisser/flink that referenced this pull request Jun 3, 2026
…nk-gs-fs-hadoop (apache#28286)

* [FLINK-39136][filesystems] Bump google-cloud-storage to 2.68.0 in flink-gs-fs-hadoop

The GCS file system bundled google-cloud-storage 2.29.1, which throws a
NullPointerException instead of retrying certain GCS 503 Service Unavailable
errors during resumable uploads, breaking checkpointing for jobs writing to
gs:// via a RecoverableWriter. The upstream fix is in googleapis/java-storage#2987.

Bump google-cloud-storage 2.29.1 -> 2.68.0 and the matching grpc artifacts
1.59.1 -> 1.81.0, regenerate the bundled-dependency NOTICE accordingly, add the
bundled license file for the newly bundled stax2-api, and update the version
links in the GCS filesystem documentation.

Add integration tests that run against a real GCS bucket, mirroring the existing
S3 filesystem integration tests. They are skipped unless a bucket is configured
via the IT_CASE_GCS_BUCKET environment variable; authentication uses Application
Default Credentials (GOOGLE_APPLICATION_CREDENTIALS).

Generated-by: Claude Code (Opus 4.8)
MartijnVisser added a commit to apache/flink that referenced this pull request Jun 4, 2026
…nk-gs-fs-hadoop (#28286)

* [FLINK-39136][filesystems] Bump google-cloud-storage to 2.68.0 in flink-gs-fs-hadoop

The GCS file system bundled google-cloud-storage 2.29.1, which throws a
NullPointerException instead of retrying certain GCS 503 Service Unavailable
errors during resumable uploads, breaking checkpointing for jobs writing to
gs:// via a RecoverableWriter. The upstream fix is in googleapis/java-storage#2987.

Bump google-cloud-storage 2.29.1 -> 2.68.0 and the matching grpc artifacts
1.59.1 -> 1.81.0, regenerate the bundled-dependency NOTICE accordingly, add the
bundled license file for the newly bundled stax2-api, and update the version
links in the GCS filesystem documentation.

Add integration tests that run against a real GCS bucket, mirroring the existing
S3 filesystem integration tests. They are skipped unless a bucket is configured
via the IT_CASE_GCS_BUCKET environment variable; authentication uses Application
Default Credentials (GOOGLE_APPLICATION_CREDENTIALS).

Generated-by: Claude Code (Opus 4.8)
MartijnVisser added a commit to apache/flink that referenced this pull request Jun 4, 2026
…nk-gs-fs-hadoop (#28286)

* [FLINK-39136][filesystems] Bump google-cloud-storage to 2.68.0 in flink-gs-fs-hadoop

The GCS file system bundled google-cloud-storage 2.29.1, which throws a
NullPointerException instead of retrying certain GCS 503 Service Unavailable
errors during resumable uploads, breaking checkpointing for jobs writing to
gs:// via a RecoverableWriter. The upstream fix is in googleapis/java-storage#2987.

Bump google-cloud-storage 2.29.1 -> 2.68.0 and the matching grpc artifacts
1.59.1 -> 1.81.0, regenerate the bundled-dependency NOTICE accordingly, add the
bundled license file for the newly bundled stax2-api, and update the version
links in the GCS filesystem documentation.

Add integration tests that run against a real GCS bucket, mirroring the existing
S3 filesystem integration tests. They are skipped unless a bucket is configured
via the IT_CASE_GCS_BUCKET environment variable; authentication uses Application
Default Credentials (GOOGLE_APPLICATION_CREDENTIALS).

Generated-by: Claude Code (Opus 4.8)
MartijnVisser added a commit to apache/flink that referenced this pull request Jun 4, 2026
…nk-gs-fs-hadoop (#28286)

* [FLINK-39136][filesystems] Bump google-cloud-storage to 2.68.0 in flink-gs-fs-hadoop

The GCS file system bundled google-cloud-storage 2.29.1, which throws a
NullPointerException instead of retrying certain GCS 503 Service Unavailable
errors during resumable uploads, breaking checkpointing for jobs writing to
gs:// via a RecoverableWriter. The upstream fix is in googleapis/java-storage#2987.

Bump google-cloud-storage 2.29.1 -> 2.68.0 and the matching grpc artifacts
1.59.1 -> 1.81.0, regenerate the bundled-dependency NOTICE accordingly, add the
bundled license file for the newly bundled stax2-api, and update the version
links in the GCS filesystem documentation.

Add integration tests that run against a real GCS bucket, mirroring the existing
S3 filesystem integration tests. They are skipped unless a bucket is configured
via the IT_CASE_GCS_BUCKET environment variable; authentication uses Application
Default Credentials (GOOGLE_APPLICATION_CREDENTIALS).

Generated-by: Claude Code (Opus 4.8)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the googleapis/java-storage API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants