Uploaded image for project: 'Couchbase .NET client library'
  1. Couchbase .NET client library
  2. NCBC-3419

Use System.Text.Json for deserializing all query response metadata

Details

    • Improvement
    • Resolution: Unresolved
    • Minor
    • None
    • None
    • library
    • None

    Description

      Currently, we use the custom ITypeSerializer to deserialize the entire query response. This can, in theory, cause problems with inconsistent deserialization of metadata. An example is NCBC-3217 where AdditionalProperies may be deserialized as either System.Text.Json.JsonElement or Newtonsoft.Json.Linq.JToken. This could be further exacerbated by a custom serializer that can't handle AdditionalProperties, struggles with our custom property name attributes, and more.

      Instead, the custom deserializer should apply only to the rows returned by the query, not any system-level metadata such as errors, metrics, etc.

      Attachments

        For Gerrit Dashboard: NCBC-3419
        # Subject Branch Project Status CR V

        Activity

          People

            btburnett3 Brant Burnett
            btburnett3 Brant Burnett
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:

              Gerrit Reviews

                PagerDuty