Uploaded image for project: 'C Driver'
  1. C Driver
  2. CDRIVER-3306

Do not assume non-empty server reply implies stream is still valid

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major - P3
    • Resolution: Fixed
    • None
    • 1.17.0-beta, 1.17.0
    • bulk, libmongoc
    • None

    Description

      UPDATE: This ticket has been cloned so that the clone will cover Scenario 1 (as described by Kevin's comment from 03/09/2020) and this original ticket will cover Scenario 2 (as described in the aforementioned comment).

      A mongoc_server_stream_t should not be reused after calling mongoc_cluster_run_command_monitored on with the stream.

      In two situations, the stream will disconnect and be invalidated:

      • a network error (example)
      • a "not master" or "node is recovering error" for pre 4.2 server (here)

      When that happens, the server_stream for that operation is no longer valid. But we do not always account that correctly.

      _mongoc_write_opmsg (incorrectly) always ends unordered bulk operations on error because of CDRIVER-3305. Once that is resolved, it will need to handle the case of an invalidated server stream.

      _mongoc_write_opquery only sets must_stop to true for what it deems is a network error (of which, that check might not be quite right if error labels were applied). But it does not handle the case of a "not master" or "node is recovering" error causing a disconnect.

      Attachments

        Issue Links

          Activity

            People

              roberto.sanchez@mongodb.com Roberto Sanchez
              kevin.albertson@mongodb.com Kevin Albertson
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: