Skip to content

feat(generator): emit ResumableUploadCallSettings on stub and service settings - #14318

Draft
whowes wants to merge 1 commit into
whowes/generator-allowlist-parserfrom
whowes/generator-settings-codegen
Draft

whowes wants to merge 1 commit into
whowes/generator-allowlist-parserfrom
whowes/generator-settings-codegen

Conversation

@whowes

@whowes whowes commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Adds generator support for emitting ResumableUploadCallSettings on stub and service settings classes to support protocol-specific upload options. Resumable upload settings are populated with global timeouts extracted from the service configuration while bypassing standard unary retries.

gemini-code-assist[bot]

This comment was marked as outdated.

@whowes
whowes added this pull request to stack #14327 September 9, 2026 06:35
@whowes
whowes force-pushed the whowes/generator-settings-codegen branch from 9e50315 to e99b1f1 Compare September 9, 2026 15:50
@whowes
whowes force-pushed the whowes/generator-settings-codegen branch from e99b1f1 to 14d6389 Compare September 9, 2026 20:16
@whowes
whowes removed this pull request from stack #14327 September 9, 2026 23:45
@whowes
whowes added this pull request to stack #14343 September 9, 2026 23:47
@whowes
whowes force-pushed the whowes/generator-settings-codegen branch from 14d6389 to 74e2bea Compare September 9, 2026 23:50
@whowes
whowes force-pushed the whowes/generator-settings-codegen branch from 74e2bea to b03ac25 Compare September 10, 2026 01:15
@whowes
whowes force-pushed the whowes/generator-settings-codegen branch from b03ac25 to fb557d9 Compare September 10, 2026 06:04
@whowes
whowes force-pushed the whowes/generator-settings-codegen branch from fb557d9 to 8f9a345 Compare September 11, 2026 17:23
@whowes
whowes removed this pull request from stack #14343 September 11, 2026 17:25
@whowes
whowes added this pull request to stack #14363 September 11, 2026 17:25
@whowes
whowes force-pushed the whowes/generator-settings-codegen branch from 8f9a345 to 2c56792 Compare September 11, 2026 20:00
@whowes
whowes force-pushed the whowes/generator-settings-codegen branch from 2c56792 to 675dd7d Compare September 11, 2026 21:13
@whowes
whowes force-pushed the whowes/generator-settings-codegen branch from 675dd7d to 6eaf35a Compare September 11, 2026 21:39
@whowes
whowes force-pushed the whowes/generator-settings-codegen branch from 6eaf35a to 8e793ed Compare September 14, 2026 21:34
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed for 'gapic-generator-java-root'

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@sonarqubecloud

Copy link
Copy Markdown

* <p>Note that custom retry settings and headers configured via ApiCallContext apply strictly
* to the initial session initiation request.
*/
public ResumableUploadCallSettings.Builder uploadMediaSettings() {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think uploadMediaSettings method in this ServiceSettings file and StubSettings below are newly generated?

/**
* Returns the builder for the settings used for calls to uploadMedia.
*
* <p>Note that custom retry settings and headers configured via ApiCallContext apply strictly

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we can configure headers via ApiCallContext in settings. It does apply to the callable since ApiCallContext is only exposed there.

@whowes
whowes removed this pull request from stack #14363 September 14, 2026 22:18
@whowes
whowes added this pull request to stack #14381 September 14, 2026 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants