Skip to content

60 * 0.8 + 2 * i should be less than retry_state#2907

Merged
ganmacs merged 1 commit into
fluent:masterfrom
ganmacs:follow-up-2901
Mar 24, 2020
Merged

60 * 0.8 + 2 * i should be less than retry_state#2907
ganmacs merged 1 commit into
fluent:masterfrom
ganmacs:follow-up-2901

Conversation

@ganmacs

@ganmacs ganmacs commented Mar 24, 2020

Copy link
Copy Markdown
Member

Which issue(s) this PR fixes:
Follow up #2901

What this PR does / why we need it:

in this test retry_state = 60. So i should be less than 9

@next_time >= @timeout_at || !!(@max_steps && @steps >= @max_steps)

Docs Changes:

no need

Release Note:

no need

in this test retry_state = 60. So `i` should be less than 9

Signed-off-by: Yuta Iwama <ganmacs@gmail.com>
@ganmacs ganmacs self-assigned this Mar 24, 2020
@ganmacs ganmacs requested review from cosmo0920 and repeatedly March 24, 2020 03:30
@ganmacs ganmacs merged commit 02d503d into fluent:master Mar 24, 2020
@ganmacs ganmacs deleted the follow-up-2901 branch March 24, 2020 05:04
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