Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 27, 2025

Addresses review feedback on #6574 regarding inconsistent use of the max_delay parameter in the exponential backoff cURL example.

Changes

  • Updated cURL example in content/shared/influxdb3-write-guides/troubleshoot-distributed.md to use $max_delay variable instead of hardcoded 30
  • Passes max_delay as awk variable -v m=$max_delay and uses it in the comparison: if(d>m) d=m
  • Makes cURL example consistent with Python and JavaScript examples which already use their respective max_delay variables

Before:

delay=$(awk -v b=$base -v a=$attempt 'BEGIN{d=b*(2^a); if(d>30) d=30; print d}')

After:

delay=$(awk -v b=$base -v a=$attempt -v m=$max_delay 'BEGIN{d=b*(2^a); if(d>m) d=m; print d}')

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: jstirnaman <212227+jstirnaman@users.noreply.github.com>
Copilot AI changed the title [WIP] Update write responses to implement $maxDelay changes Fix cURL example to use $max_delay variable instead of hardcoded value Nov 27, 2025
Copilot AI requested a review from jstirnaman November 27, 2025 15:12
Copilot finished work on behalf of jstirnaman November 27, 2025 15:12
@jstirnaman jstirnaman marked this pull request as ready for review November 28, 2025 23:39
@jstirnaman jstirnaman merged commit 46eb577 into dist-write-resp Nov 28, 2025
3 checks passed
@jstirnaman jstirnaman deleted the copilot/sub-pr-6574 branch November 28, 2025 23:40
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