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

Better error grouping key for opentelemetry SDKs #299

Open
lahsivjar opened this issue Jun 20, 2024 · 0 comments
Open

Better error grouping key for opentelemetry SDKs #299

lahsivjar opened this issue Jun 20, 2024 · 0 comments

Comments

@lahsivjar
Copy link
Contributor

Current logic for error grouping key prioritizes exception.type, log.param_message (this is non formatted error string), and the stacktrace. The final fallback is to use log.message which is formatted error string. This logic ensures that we contain the cardinality of the grouping key, however, with OTel SDKs we don't get stacktrace and log.param_message. This makes the grouping key rely only on exception.type taking away a lot of the usability of the grouping.

One possible solution might be to use the log.message if we can't get log.param_message OR the stacktrace. We can contain the cardinality explosion by a locality sensitive hashing techniques or heuristics (based on tokenization of the message and removing fields that are parameterized for example URIs or numbers).

NOTE: Current evidences are collected ONLY using opentelemetry-go and we may want to check some other language SDKs for a holistic view.

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

No branches or pull requests

1 participant