[pull] master from DataDog:master - #651
Merged
Merged
Conversation
…24471) * kafka_consumer: store broker_timestamps as marshal instead of json The DSM broker_timestamps cache was serialized with json.dumps on every check run. At high partition counts json.dumps is the dominant allocation source (profiler flagged _save_broker_timestamps -> json.dumps -> iterencode as the top allocator, feeding glibc arena fragmentation and CCR OOMs, #incident-57455). Persist with marshal (base64-wrapped to ride the string persistent-cache API) instead. marshal is a compact binary codec that avoids JSON's per-number text encoding, cutting serialization ~100x on the save path with no change to emitted metrics. The on-disk format changes and is version-specific, so a cache written by an older agent is discarded once (rebuilt next run). Tests seed and read the cache in the marshal format with integer offset keys. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> * Add changelog entry Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
* feat(incident-io): add account_config.json with API key auth * Trigger CODEOWNERS re-evaluation * Update label for API key in account_config.json * Update account_config.json
* cockroachdb: fix stale metric names in metadata.csv METRIC_MAP renames the six changefeed_*_hist_nanos raw metric families by dropping the _nanos suffix (e.g. changefeed.checkpoint_hist), but metadata.csv still listed the old _nanos-suffixed names for the bucket/count/sum rows, and had no entries for go.scheduler_latency. Add a test that runs without histogram_buckets_as_distributions, which is what surfaces the mismatch since the default instance bypasses classic bucket/count/sum submission entirely. Environment: Datadog workspace Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * Add changelog entry for #24461 Environment: Datadog workspace Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * Remove unneeded changelog entry for #24461 metadata.csv is not shipped with the Agent, so no changelog entry is required for this change. Environment: Datadog workspace Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )