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

DRIVERS-1389 Make ExceededTimeLimit a retryable reads error #1478

Merged
merged 9 commits into from
Nov 30, 2023
Merged

DRIVERS-1389 Make ExceededTimeLimit a retryable reads error #1478

merged 9 commits into from
Nov 30, 2023

Commits on Nov 28, 2023

  1. Add unified tests for making ExceededTimeLimit a retryable error

    @kkloberdanz
    kkloberdanz committed Nov 28, 2023
    Configuration menu
    Copy the full SHA
    2e4a2e2 View commit details
    Browse the repository at this point in the history

  2. update retryable reads spec to add ExceededTimeLimit to the list of e… 

    …rror codes that should receive a RetryableReadError label
    @kkloberdanz
    kkloberdanz committed Nov 28, 2023
    Configuration menu
    Copy the full SHA
    7641533 View commit details
    Browse the repository at this point in the history

  3. fix schema version, use 1.3 instead of 1.0

    @kkloberdanz
    kkloberdanz committed Nov 28, 2023
    Configuration menu
    Copy the full SHA
    550f756 View commit details
    Browse the repository at this point in the history

Commits on Nov 29, 2023

  1. Update source/retryable-reads/tests/unified/ExceededTimeLimit.yml 

    Co-authored-by: Jeremy Mikola <jmikola@gmail.com>
    @kkloberdanz @jmikola
    kkloberdanz and jmikola committed Nov 29, 2023
    Configuration menu
    Copy the full SHA
    5a2c3fd View commit details
    Browse the repository at this point in the history

  2. implement Jeremy's suggestions

    @kkloberdanz
    kkloberdanz committed Nov 29, 2023
    Configuration menu
    Copy the full SHA
    6718a44 View commit details
    Browse the repository at this point in the history

  3. use 4.1.7 for sharded and load-balanced clusters, 4.0 otherwise

    @kkloberdanz
    kkloberdanz committed Nov 29, 2023
    Configuration menu
    Copy the full SHA
    61faa8f View commit details
    Browse the repository at this point in the history

  4. Update source/retryable-reads/retryable-reads.rst 

    Co-authored-by: Kevin Albertson <kevin.albertson@10gen.com>
    @kkloberdanz @kevinAlbs
    kkloberdanz and kevinAlbs committed Nov 29, 2023
    Configuration menu
    Copy the full SHA
    820b2d0 View commit details
    Browse the repository at this point in the history

  5. implement Kevin's suggestions

    @kkloberdanz
    kkloberdanz committed Nov 29, 2023
    Configuration menu
    Copy the full SHA
    03acef6 View commit details
    Browse the repository at this point in the history

Commits on Nov 30, 2023

  1. Update source/retryable-reads/tests/unified/exceededTimeLimit.yml 

    Co-authored-by: Jeremy Mikola <jmikola@gmail.com>
    @kkloberdanz @jmikola
    kkloberdanz and jmikola committed Nov 30, 2023
    Configuration menu
    Copy the full SHA
    2412f5f View commit details
    Browse the repository at this point in the history