Uploaded image for project: 'Couchbase C client library libcouchbase'
  1. Couchbase C client library libcouchbase
  2. CCBC-1441

Update documentation to use lcb_cmd{query,analytics}_positional_params

    XMLWordPrintable

Details

    • Task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 3.2.0
    • 3.2.2
    • docs

    Description

      The new lcb_cmdquery_positional_params() and lcb_cmdanalytics_positional_params() should be preferred to their singular alternatives (lcb_cmdquery_positional_param() and lcb_cmdanalytics_positional_param())

      The singular functions behavior was inadvertently broken in LCB 3.2.0, and is being reverted in 3.2.1. In addition, LCB 3.2.1 has introduced new plural forms of the functions which accept all positional parameters as single JSON-encoded array.

      The previous (singular) API does not allow arguments to be replaced by index or replaced entirely without resetting the entire command. The new (plural) API just overwrites the "args" property in the payload and therefore has a more predictable behavior and is easier to use.

      Attachments

        Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            hakim.cassimally Hakim Cassimally added a comment - PR https://github.com/couchbase/docs-sdk-c/pull/112

            As per Ray Cardillo's comment, will check after 3.2.1 build is complete and merge then if all ok.

            hakim.cassimally Hakim Cassimally added a comment - As per Ray Cardillo 's comment, will check after 3.2.1 build is complete and merge then if all ok.

            And 3.2.1 was done on Friday, looks good, merging.

            hakim.cassimally Hakim Cassimally added a comment - And 3.2.1 was done on Friday, looks good, merging.

            People

              hakim.cassimally Hakim Cassimally
              avsej Sergey Avseyev
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty