Skip to content

feat(codegen): add defaultEnableSocketTimeout2026 customization config field - #7348

Merged
zoewangg merged 1 commit into
masterfrom
zoewang/phase0-noop-default-socket-timeout-2026
Sep 4, 2026
Merged

feat(codegen): add defaultEnableSocketTimeout2026 customization config field#7348
zoewangg merged 1 commit into
masterfrom
zoewang/phase0-noop-default-socket-timeout-2026

Conversation

@zoewangg

@zoewangg zoewangg commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Motivation and Context

This adds a new customization-config field, defaultEnableSocketTimeout2026, to the codegen customization model. It is additive and inert: the field is recognized when a service's customization.config is parsed, but nothing in codegen consumes it, so no generated code or runtime behavior changes.

Modifications

  • Added a Boolean defaultEnableSocketTimeout2026 field (with Javadoc) and its getter/setter to CustomizationConfig, mirroring the existing defaultNewRetries2026 field.
  • No consumer: the field is read nowhere in codegen, so with no service setting it the generated output is unchanged. Default value is null (off).

Testing

  • mvn install -pl :codegen -P quick (compile) succeeds.
  • No dedicated test added: the change is a no-op field with no behavior to exercise, matching the defaultNewRetries2026 precedent (no dedicated test for that field either).

Screenshots (if appropriate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist

  • I have read the CONTRIBUTING document
  • Local run of mvn install succeeds
  • My code follows the code style of this project
  • My change requires a change to the Javadoc documentation
  • I have updated the Javadoc documentation accordingly
  • I have added tests to cover my changes
  • All new and existing tests passed
  • I have added a changelog entry. Adding a new entry must be accomplished by running the scripts/new-change script and following the instructions. Commit the new file created by the script in .changes/next-release with your changes.
  • My change is to implement 1.11 parity feature and I have updated LaunchChangelog

License

  • I confirm that this pull request can be released under the Apache 2 license

Add a no-op Boolean field to the codegen customization config model
so customization.config files carrying this key parse without error.
Nothing consumes it yet, so no generated code or behavior changes.
Prepares for the upcoming default socket-timeout feature.
@zoewangg
zoewangg requested a review from a team as a code owner September 3, 2026 21:15
@zoewangg zoewangg added the changelog-not-required Indicate changelog entry is not required for a specific PR label Sep 3, 2026
@zoewangg
zoewangg requested a review from joviegas September 3, 2026 21:19
@zoewangg
zoewangg enabled auto-merge September 3, 2026 21:50
@zoewangg
zoewangg added this pull request to the merge queue Sep 3, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Sep 4, 2026
@zoewangg
zoewangg added this pull request to the merge queue Sep 4, 2026
Merged via the queue into master with commit 1bf4baa Sep 4, 2026
20 of 22 checks passed
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

This pull request has been closed and the conversation has been locked. Comments on closed PRs are hard for our team to see. If you need more assistance, please open a new issue that references this one.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 4, 2026
@zoewangg
zoewangg deleted the zoewang/phase0-noop-default-socket-timeout-2026 branch September 4, 2026 21:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

changelog-not-required Indicate changelog entry is not required for a specific PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants