Skip to content

Clean up witness logging - #1170

Merged
AlCutter merged 1 commit into
mainfrom
clean_log
Sep 10, 2026
Merged

AlCutter merged 1 commit into
mainfrom
clean_log

Conversation

@AlCutter

Copy link
Copy Markdown
Collaborator

Improve logging for witnessing.

@AlCutter
AlCutter requested a review from a team as a code owner September 10, 2026 11:23
@AlCutter
AlCutter requested a review from phbnf September 10, 2026 11:23
@AlCutter AlCutter changed the title Clean up witness logging. Clean up witness logging Sep 10, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.50.

Benchmark suite Current: e1fbf08 Previous: 8977b14 Ratio
BenchmarkDedup (github.com/transparency-dev/tessera) 4174572 ns/op 904284 B/op 18804 allocs/op 1777569 ns/op 887701 B/op 18666 allocs/op 2.35
BenchmarkDedup (github.com/transparency-dev/tessera) - ns/op 4174572 ns/op 1777569 ns/op 2.35
BenchmarkLeafBundle_UnmarshalText (github.com/transparency-dev/tessera/api) 8933 ns/op 6528 B/op 1 allocs/op 1443 ns/op 6528 B/op 1 allocs/op 6.19
BenchmarkLeafBundle_UnmarshalText (github.com/transparency-dev/tessera/api) - ns/op 8933 ns/op 1443 ns/op 6.19
BenchmarkProofBuilder/InclusionProof/WarmCache (github.com/transparency-dev/tessera/client) 118157 ns/op 15352 B/op 180 allocs/op 49401 ns/op 15326 B/op 180 allocs/op 2.39
BenchmarkProofBuilder/InclusionProof/WarmCache (github.com/transparency-dev/tessera/client) - ns/op 118157 ns/op 49401 ns/op 2.39
BenchmarkProofBuilder/InclusionProof/ColdCache (github.com/transparency-dev/tessera/client) 1096854 ns/op 689101 B/op 3555 allocs/op 517302 ns/op 689120 B/op 3555 allocs/op 2.12
BenchmarkProofBuilder/InclusionProof/ColdCache (github.com/transparency-dev/tessera/client) - ns/op 1096854 ns/op 517302 ns/op 2.12

This comment was automatically generated by workflow using github-action-benchmark.

@AlCutter
AlCutter merged commit 53d63f1 into main Sep 10, 2026
27 checks passed
@AlCutter
AlCutter deleted the clean_log branch September 10, 2026 11:34
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.

2 participants