Fix successive failures error in streamer by timvaillancourt · Pull Request #1371 · github/gh-ost

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@timvaillancourt

Copy link Copy Markdown

Collaborator

Related issue: #1370

Description

This PR fixes a conditional in go/logic/streamer.go so the specified max retries is respected

In case this PR introduced Go code changes:

  • contributed code is using same conventions as original code
  • script/cibuild returns with no formatting errors, build errors or unit test errors.
Signed-off-by: Tim Vaillancourt <tim@timvaillancourt.com>

@timvaillancourt timvaillancourt requested a review from a user

January 22, 2024 23:28

@timvaillancourt timvaillancourt added this to the v1.1.7 milestone

Jan 22, 2024

@timvaillancourt timvaillancourt linked an issue

Jan 22, 2024

that may be closed by this pull request

@timvaillancourt timvaillancourt merged commit 11d66d8 into github:master

Jan 23, 2024

@timvaillancourt timvaillancourt deleted the fix-streamer-maxRetries-err branch

January 23, 2024 18:46

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

Reviewers

@meiji163 meiji163 meiji163 approved these changes

@rashiq rashiq Awaiting requested review from rashiq rashiq is a code owner

Assignees

No one assigned

Projects

None yet

Milestone

v1.1.7

Development

Successfully merging this pull request may close these issues.

Incorrect max retries in streamer

2 participants

@timvaillancourt @meiji163