Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-42412

Query is reading from opaque field "txdata" which will introduce build instability

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Major
    • None
    • None
    • query
    • 1

    Description

      We have previously agreed that BEGIN WORK "txdata" field is opaque to query, and will be passed directly from SDK to gocbcore without interpretation or reading.

      However I can see this field is being read from here: https://github.com/couchbase/query/blob/ce919de4316e2b24a7118d0dec5ace910fed789e/execution/txauthorize.go#L58

      Please remove this code as soon as possible.  That is an unstable work-in-progress interface that we will be making changes to in the near future, and this query code will introduce build instabilities as a result.

      As agreed elsewhere we will give you gocbcore APIs for everything you're using there (except KvDurableTimeoutMs and KvTimeoutMs, as also explained elsewhere) but please do not read the "txdata" field directly.

      Attachments

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

        Activity

          People

            kamini.jagtiani Kamini Jagtiani
            graham.pople Graham Pople
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty