Fix inconsistent wording for prose retryable writes test 6, case 3

XMLWordPrintableJSON

    • Type: Spec Change
    • Resolution: Unresolved
    • Priority: Unknown
    • None
    • Component/s: Backpressure
    • None
    • Not Needed
    • Hide

      Summary of necessary driver changes

      • Drivers may need to update the affected prose test for consistency.

      Commits for syncing spec/prose tests
      (and/or refer to an existing language POC if needed)

      Context for other referenced/linked tickets

      •  
      Show
      Summary of necessary driver changes Drivers may need to update the affected prose test for consistency. Commits for syncing spec/prose tests (and/or refer to an existing language POC if needed) cadc0726cd5e3a201ede910e405b0731f6830cf5 Context for other referenced/linked tickets  
    • $i18n.getText("admin.common.words.hide")
      Key Status/Resolution FixVersion
      CDRIVER-6279 Done 2.3.0
      CXX-3455 Backlog
      CSHARP-5955 Done
      GODRIVER-3850 Done
      JAVA-6157 Done 5.7.0
      NODE-7508 Fixed 7.2.0
      PYTHON-5771 Works as Designed
      PHPLIB-1822 Backlog
      RUBY-3799 Done
      RUST-2393 Works as Designed
      $i18n.getText("admin.common.words.show")
      #scriptField, #scriptField *{ border: 1px solid black; } #scriptField{ border-collapse: collapse; } #scriptField td { text-align: center; /* Center-align text in table cells */ } #scriptField td.key { text-align: left; /* Left-align text in the Key column */ } #scriptField a { text-decoration: none; /* Remove underlines from links */ border: none; /* Remove border from links */ } /* Add green background color to cells with FixVersion */ #scriptField td.hasFixVersion { background-color: #00FF00; /* Green color code */ } #scriptField td.willNotDo { background-color: #FF0000; /* Red color code */ } /* Center-align the first row headers */ #scriptField th { text-align: center; } Key Status/Resolution FixVersion CDRIVER-6279 Done 2.3.0 CXX-3455 Backlog CSHARP-5955 Done GODRIVER-3850 Done JAVA-6157 Done 5.7.0 NODE-7508 Fixed 7.2.0 PYTHON-5771 Works as Designed PHPLIB-1822 Backlog RUBY-3799 Done RUST-2393 Works as Designed

      6. Test error propagation after encountering multiple errors.:Case 3: Test that drivers return the correct error when receiving some errors with `NoWritesPerformed` and some without `NoWritesPerformed` has the following issues that should be fixed:

      1. It instructs to configure two fail points with error code 91 (NotWritablePrimary) and 91 (ShutdownInProgress). Given that the error codes specified are the same, but names specified are different, it is unclear whether it was meant to instruct error codes to be 91 and 91, or 10107 and 91.
        1. The intended requirement was to specify 91 and 91.
      2. It instructs to configure monitorCommands=true, but that is not an option recognized in the specification.
        1. Cases 1 and 2 just say "Via the command monitoring", case 3 should use the same wording.
      3. Cases 1 and 2 instruct to configure a fail point in step 2, and in step 3 instruct to configure another fail point via a command listener. Such narrative order is consistent with the causal order. Case 3 does it differently: step 2 instructs to configure a fail point via a command listener, while step 3 instructs to configure one normally. Not only the order chosen in case 3 is inconsistent with the causal order, but also makes it more difficult for a reader to see how case 3 is similar to cases 1 and 2.
        1. Case 3 should switch steps 2 and 3 places to match the narrative order chosen in cases 1 and 2.

            Assignee:
            Noah Stapp
            Reporter:
            Valentin Kavalenka
            Daria Pardue Daria Pardue
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: