Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moving retry interval to ApplicationFailure per Yimins request. #344

Merged
merged 2 commits into from
Jan 12, 2024

Conversation

ast2023
Copy link
Contributor

@ast2023 ast2023 commented Jan 12, 2024

What changed?
retry delay interval was moved from ActivityFailed message to ApplicationFailure

Why?
Requested by Yimin

No

@ast2023 ast2023 requested review from a team as code owners January 12, 2024 18:19
@ast2023 ast2023 enabled auto-merge (squash) January 12, 2024 18:19
Comment on lines 1 to 2
// The MIT License
//
Copy link
Member

Choose a reason for hiding this comment

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

Accidentally removed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you!

Copy link
Member

@cretz cretz left a comment

Choose a reason for hiding this comment

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

Since you're technically making a backwards incompatible change to our API, you'll have to disable the buf compatibility checker in this PR and re-enable it in another PR after you've merged this one. Right now CI will fail.

(marking approved as agreed, though I don't personally agree w/ this being on the error)

@ast2023 ast2023 merged commit 6f67ff7 into master Jan 12, 2024
3 checks passed
@ast2023 ast2023 deleted the sasha/api-update branch January 12, 2024 21:55
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.

None yet

3 participants