Skip to content

@W-21570079 Track request latency metrics that exposes tool data#263

Merged
jarhun88 merged 19 commits intomainfrom
dev/feat/latencymetric
Mar 17, 2026
Merged

@W-21570079 Track request latency metrics that exposes tool data#263
jarhun88 merged 19 commits intomainfrom
dev/feat/latencymetric

Conversation

@jarhun88
Copy link
Contributor

@jarhun88 jarhun88 commented Mar 12, 2026

IMPORTANT: Please do not create a Pull Request without creating an issue first.

Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of
the pull request.

Pull Request Template

Description

creating a way to track request latency for monitoring purposes

Here is what the latency metric looks like:

[2026-03-12T23:27:40.861Z] [stdout] �[32m'{\n'�[39m +
  �[32m'  "tags": {\n'�[39m +
  �[32m'    "agent_language": "nodejs",\n'�[39m +
  �[32m'    "agent_version": "2.3.1",\n'�[39m +
  �[32m'    "device": "jsong-ltmrqfc.internal.salesforce.com",\n'�[39m +
  �[32m'    "env": "dev",\n'�[39m +
  �[32m'    "process_runtime_version": "22.15.0",\n'�[39m +
  �[32m'    "process_runtime_name": "nodejs",\n'�[39m +
  �[32m'    "http_request_method": "post",\n'�[39m +
  �[32m'    "http_route": "_tableau-mcp",\n'�[39m +
  �[32m'    "http_response_status_code": "200",\n'�[39m +
  �[32m'    "tool_name": "list-datasources",\n'�[39m +
  �[32m'    "server": "https___test-dataplane7_tableau_sfdc-ckzqgc_svc_sfdcfc_net",\n'�[39m +
  �[32m'    "site_id": "3d2acf7b-8dd6-4afc-9a86-6a1eb2a1b525"\n'�[39m +
  �[32m'  },\n'�[39m +
  �[32m'  "service": "tableau-mcp",\n'�[39m +
  �[32m'  "subservice": "",\n'�[39m +
  �[32m'  "scope": [\n'�[39m +
  �[32m'    "my-org",\n'�[39m +
  �[32m'    "my-team",\n'�[39m +
  �[32m'    "my-app"\n'�[39m +
  �[32m'  ],\n'�[39m +
  �[32m'  "timestamp": 1773358060857,\n'�[39m +
  �[32m'  "metricName": [\n'�[39m +
  �[32m'    "apm_apm_http_server_request_duration"\n'�[39m +
  �[32m'  ],\n'�[39m +
  �[32m'  "metricValue": 290.44022692499965\n'�[39m +
  �[32m'}

Motivation and Context

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

How Has This Been Tested?

Related Issues

Checklist

  • I have updated the version in the package.json file by using npm run version. For example,
    use npm run version:patch for a patch version bump.
  • I have made any necessary changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have documented any breaking changes in the PR description. For example, renaming a config
    environment variable or changing its default value.

Contributor Agreement

By submitting this pull request, I confirm that:

Copy link
Collaborator

@anyoung-tableau anyoung-tableau left a comment

Choose a reason for hiding this comment

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

💪💪💪

@jarhun88 jarhun88 merged commit c260404 into main Mar 17, 2026
6 checks passed
@jarhun88 jarhun88 deleted the dev/feat/latencymetric branch March 17, 2026 23:50
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