Skip to content

[reorg fix] [DOCS-15299] Document Flutter assignment request limits - #38870

Closed
jhgilbert wants to merge 2 commits into
masterfrom
reorg-fix/pr-38720
Closed

[reorg fix] [DOCS-15299] Document Flutter assignment request limits#38870
jhgilbert wants to merge 2 commits into
masterfrom
reorg-fix/pr-38720

Conversation

@jhgilbert

Copy link
Copy Markdown
Collaborator

🤖 Auto-generated fix for #38720.

@leoromanovsky — the docs repo reorg created merge conflicts in your PR #38720. This is an auto-generated replacement with the file paths fixed; please use it instead of the original.

This PR replays the commits from #38720 with file paths translated to the post-reorg hugo/ layout. The original commits are preserved — same messages and authorship.

The original PR (#38720) will be closed in favor of this one.

Next steps:

  1. Verify that this PR looks correct in the browser.
  2. Remove the WORK IN PROGRESS label from this PR.
  3. Wait for the standard docs team approval before merging. Optionally, you can check the 'ready for merge' checkbox below if you would like the docs team to merge it for you.
  • Ready for merge

Original PR description:

Motivation

Fixes DOCS-15299.

Users need the default Flutter assignment request limits and the controls for advanced use cases.

Changes and Decisions

Document the one-second timeout per attempt and one default retry. Add an override example with assignmentRequestTimeout and assignmentRequestRetryCount. Keep the settings in the existing advanced configuration section.

Merge instructions

Merge readiness:

  • Ready for merge

@jhgilbert
jhgilbert requested a review from a team as a code owner August 6, 2026 14:22
@jhgilbert jhgilbert added WORK IN PROGRESS No review needed, it's a wip ;) astro-reorg-auto-pr Needs manual conflict resolution after replatforming reorg labels Aug 6, 2026
@datadog-datadog-prod-us1

This comment has been minimized.

@clreaume clreaume self-assigned this Aug 11, 2026
@clreaume clreaume added editorial review Waiting on a more in-depth review and removed WORK IN PROGRESS No review needed, it's a wip ;) labels Aug 11, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

: The interval at which aggregated flag evaluation telemetry is sent to Datadog. Accepted values are between 1 and 60 seconds. The default is 10 seconds.

`assignmentRequestTimeout`
: The timeout for each assignment request attempt. The default is one second. Values less than or equal to zero use the default.

@clreaume clreaume Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
: The timeout for each assignment request attempt. The default is one second. Values less than or equal to zero use the default.
: The timeout for each assignment request attempt. The default is `1` second. Values less than or equal to `0` use the default.

: The timeout for each assignment request attempt. The default is one second. Values less than or equal to zero use the default.

`assignmentRequestRetryCount`
: The number of retries after the first attempt. The default is `1`. Set this option to `0` to disable retries. Negative values are treated as `0`.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
: The number of retries after the first attempt. The default is `1`. Set this option to `0` to disable retries. Negative values are treated as `0`.
: The number of transient failure retries after the first attempt. The default is `1`. Set this option to `0` to disable retries. Negative values are treated as `0`. Transient failures include network errors, timeouts, HTTP 408, HTTP 429, and HTTP 5xx responses.

Comment on lines +301 to +302
By default, each assignment request attempt has a one-second timeout, and the SDK retries transient failures once. Transient failures include network errors, timeouts, HTTP 408, HTTP 429, and HTTP 5xx responses.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
By default, each assignment request attempt has a one-second timeout, and the SDK retries transient failures once. Transient failures include network errors, timeouts, HTTP 408, HTTP 429, and HTTP 5xx responses.

unnecessary to have a paragraph dedicated to these two; moved some of the transient failure language below

trackExposures: true,
trackEvaluations: true,
evaluationFlushInterval: const Duration(seconds: 10),
assignmentRequestTimeout: const Duration(seconds: 2),

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

is this const Duration(seconds: 2) construction accurate, or does it just take an integer/float?

I'm wondering if Claude just copied the pattern from the evaluationFlushInterval value above it

@clreaume clreaume left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

hi @leoromanovsky, left some suggestions/comments! some of them will feel familiar to comments on the other languages' PRs

@leoromanovsky

Copy link
Copy Markdown
Contributor

Superseded by #39560, which documents the finalized cross-platform assignment request timeout/retry contract in one unified PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astro-reorg-auto-pr Needs manual conflict resolution after replatforming reorg editorial review Waiting on a more in-depth review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants