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.

Sign up for GitHub

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

Jump to bottom

Update to cancel downstream operation in TimeoutStrategy.Pessimistic #1697

Merged
merged 5 commits into from
Oct 17, 2023
Merged

Update to cancel downstream operation in TimeoutStrategy.Pessimistic #1697

merged 5 commits into from
Oct 17, 2023

Commits on Oct 13, 2023

  1. Update to extend the lifetime of the combinedTokenSource so that it w… 

    …ill cancel downstream operations in TimeoutStrategy.Pessimistic.
    @lor1mp
    lor1mp committed Oct 13, 2023
    Configuration menu
    Copy the full SHA
    66b8bdb View commit details
    Browse the repository at this point in the history

Commits on Oct 16, 2023

  1. Refactor as per PR comments; cancel in finally if token hasn't been c… 

    …anceled.
    @lor1mp
    lor1mp committed Oct 16, 2023
    Configuration menu
    Copy the full SHA
    43a0d0d View commit details
    Browse the repository at this point in the history

  2. Update as per PR comments.

    @lor1mp
    lor1mp committed Oct 16, 2023
    Configuration menu
    Copy the full SHA
    9f5ed16 View commit details
    Browse the repository at this point in the history

  3. Adjust as per PR comments; remove unnecessary wait time.

    @lor1mp
    lor1mp committed Oct 16, 2023
    Configuration menu
    Copy the full SHA
    e249c03 View commit details
    Browse the repository at this point in the history

Commits on Oct 17, 2023

  1. Update comment 

    Fix wording and add link to issue.
    @martincostello
    martincostello authored Oct 17, 2023
    Configuration menu
    Copy the full SHA
    9a36f38 View commit details
    Browse the repository at this point in the history