Skip to content

Conversation

@brunotm
Copy link
Collaborator

@brunotm brunotm commented Sep 10, 2025

Only emit Outcome and report telemetry for rounds that transmit reports

use github.com/goccy/go-json in the pipeline task.bridge and task.jsonparse.
github.com/goccy/go-json is a drop-in replacement for the stdlib json package that improves the encode/decode execution time and number of memory allocations. This significantly improves the GC CPU time for the nodes running Streams with hundreds of thousands of pipeline executions per second.

Rework observation cache expiration to store absolute times

@brunotm brunotm added the build-publish Build and Publish image to SDLC label Sep 10, 2025
@brunotm brunotm force-pushed the bm/DS-892-observation-loop branch 9 times, most recently from 4417b18 to 71ac8a2 Compare September 17, 2025 14:33
@brunotm brunotm force-pushed the bm/DS-892-observation-loop branch 9 times, most recently from 3130109 to 4f8d690 Compare October 1, 2025 18:33
@brunotm brunotm marked this pull request as ready for review October 1, 2025 18:33
@brunotm brunotm requested review from a team as code owners October 1, 2025 18:33
@brunotm brunotm force-pushed the bm/DS-892-observation-loop branch from 4f8d690 to 50d679e Compare October 1, 2025 18:48
@brunotm brunotm removed the build-publish Build and Publish image to SDLC label Oct 2, 2025
Fletch153
Fletch153 previously approved these changes Oct 3, 2025
ro-tex
ro-tex previously approved these changes Oct 3, 2025
@brunotm brunotm requested a review from bolekk October 3, 2025 09:09
@brunotm brunotm dismissed stale reviews from ro-tex and Fletch153 via a617d8d October 3, 2025 09:12
@brunotm brunotm force-pushed the bm/DS-892-observation-loop branch 4 times, most recently from 5c7b4c2 to 874b69b Compare October 3, 2025 12:16
@ro-tex ro-tex enabled auto-merge October 3, 2025 15:50
github.com/goccy/go-json is a drop-in replacement for the stdlib json
package that improves in encode/decode execution time and memory
allocations.

This significantly improves the GC CPU time for the nodes running Streams
with hundreds of thousands of pipeline executions per second.
@brunotm brunotm force-pushed the bm/DS-892-observation-loop branch from 874b69b to b201b45 Compare October 3, 2025 16:06
@cl-sonarqube-production
Copy link

@ro-tex ro-tex added this pull request to the merge queue Oct 3, 2025
Merged via the queue into develop with commit 6f9d964 Oct 3, 2025
195 checks passed
@ro-tex ro-tex deleted the bm/DS-892-observation-loop branch October 3, 2025 16:47
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.

4 participants