Repository navigation
Reduce SQS body propagation parsing overhead - #12568
gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit into
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
More details
The streaming parser keeps the prior extraction behavior and does not build trees for unrelated JSON fields. The static review found no concrete defect.
🤖 Datadog Autotest · Commit 1ca5b9c · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
🟢 Java Benchmark SLOs — All performance SLOs passed
PR vs. master results
Commit: Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion. |
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
Build pipeline has failing jobs for 46f9f03: What to do next?
DetailsSince those jobs are not marked as being allowed to fail, the pipeline will most likely fail. |
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
DDCI pipeline didn't start (sourcing_failed)... Please retry. DDCI Change Request: |
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
|
5a8142c
into
master
What Does This Do
Extract propagation context from AWS SDK v1 SQS message bodies without building a JSON tree for the entire envelope.
Benchmark
Standalone JMH 1.37 on Temurin 17.0.18, Apple M4 Max, using Jackson 2.6.6:
Small-envelope allocation decreased from 1,976 to 992 B/op; timing confidence intervals overlap, so no speedup is claimed for that case.
Benchmark sources and raw results are retained locally and are not included in this PR.
Motivation
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any other useful labelsclose,fix, or any linking keywords when referencing an issueUse
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]