Skip to content

CXXCBC-632: Do not assume Analytics will always send meta fields in response#699

Merged
avsej merged 1 commit intocouchbase:mainfrom
avsej:fix-analytics
Nov 22, 2024
Merged

CXXCBC-632: Do not assume Analytics will always send meta fields in response#699
avsej merged 1 commit intocouchbase:mainfrom
avsej:fix-analytics

Conversation

@avsej
Copy link
Copy Markdown
Member

@avsej avsej commented Nov 22, 2024

"requestID", "clientContextID" and "metrics" might be missing.

For example, the following payload might be sent with status 503:

{
  "errors": [
    {
      "code": 23000,
      "msg": "Analytics Service is temporarily unavailable"
    }
  ],
  "status": "fatal"
}

…espnse

"requestID", "clientContextID" and "metrics" might be missing.

For example, the following payload might be sent with status 503:

    {
      "errors": [
        {
          "code": 23000,
          "msg": "Analytics Service is temporarily unavailable"
        }
      ],
      "status": "fatal"
    }
@avsej avsej requested review from a team and thejcfactor November 22, 2024 00:58
Copy link
Copy Markdown
Contributor

@thejcfactor thejcfactor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@avsej avsej merged commit a0d41a7 into couchbase:main Nov 22, 2024
@avsej avsej deleted the fix-analytics branch November 22, 2024 01:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants