Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Utilize telemetry to report exceptions #4867

Open
breezewish opened this issue May 11, 2022 · 0 comments
Open

Utilize telemetry to report exceptions #4867

breezewish opened this issue May 11, 2022 · 0 comments
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@breezewish
Copy link
Member

breezewish commented May 11, 2022

Enhancement

Currently there is no telemetry in TiFlash yet. Besides collecting usages by telemetry, we can also use it to catch exceptions like RUNTIME_CHECK and RUNTIME_ASSERT introduced by #4398. In this way, we can be noticed about logic errors at very early stage.

The telemetry stack looks to be easy to build up on sentry.

Notice that, unlike logs, we should only report call stack in the telemetry, in order to avoid collecting sensitive data.

@breezewish breezewish added the type/enhancement The issue or PR belongs to an enhancement. label May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Development

No branches or pull requests

1 participant