Qpid
  1. Qpid
  2. QPID-3115

Differences in connection options between c++ and python clients

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.8
    • Fix Version/s: 0.9, 0.10
    • Component/s: C++ Client
    • Labels:
      None

      Description

      For example, to set the reconnect limit, the syntax is the following:

      in C++: reconnect-limit
      in Python: reconnect_limit

      The documentation uses the Python syntax but does not call out the C++
      differences.

      To compound the problem, unrecognized options in C++ are silently ignored. If
      a user chooses the wrong syntax (as he would do if he read the documentation),
      the option does not take effect and the default is used instead, leading to
      unpredictable and undesired results.

        Activity

        Hide
        Justin Ross added a comment -

        Approved for 0.10. This was also reviewed by Steve Huston on the list.

        Show
        Justin Ross added a comment - Approved for 0.10. This was also reviewed by Steve Huston on the list.
        Hide
        Gordon Sim added a comment -

        The c++ client now recognises options via the same names as used in python (generally this means it recognises '_' in place of '-'. The old names are also still recognised. Additionally however the client will now thro exceptions when encountering unrecognised options, rather than just ignoring them.

        Show
        Gordon Sim added a comment - The c++ client now recognises options via the same names as used in python (generally this means it recognises '_' in place of '-'. The old names are also still recognised. Additionally however the client will now thro exceptions when encountering unrecognised options, rather than just ignoring them.

          People

          • Assignee:
            Gordon Sim
            Reporter:
            Gordon Sim
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development