Skip to content

Conversation

@Ziinc
Copy link
Contributor

@Ziinc Ziinc commented Jan 5, 2026

Negligible ips change, but improvement in memory and reductions by ~21% and ~28% respectively.

##### With input 1000 events #####
Name                                                                              ips        average  deviation         median         99th %
add_to_table - 100 chunk, no_get_tid=true, check_queue_size=false              4.78 K      209.39 μs     ±5.25%      209.63 μs      241.49 μs
add_to_table - 100 chunk, no_get_tid=true, check_queue_size=true               4.73 K      211.55 μs     ±6.46%      211.38 μs      245.17 μs
add_to_table - 100 chunk size, no_get_tid=false, check_queue_size=false        4.71 K      212.38 μs     ±7.33%      212.04 μs      254.47 μs

Comparison: 
add_to_table - 100 chunk, no_get_tid=true, check_queue_size=false              4.78 K
add_to_table - 100 chunk, no_get_tid=true, check_queue_size=true               4.73 K - 1.01x slower +2.16 μs
add_to_table - 100 chunk size, no_get_tid=false, check_queue_size=false        4.71 K - 1.01x slower +2.99 μs

Memory usage statistics:

Name                                                                       Memory usage
add_to_table - 100 chunk, no_get_tid=true, check_queue_size=false             100.19 KB
add_to_table - 100 chunk, no_get_tid=true, check_queue_size=true              100.19 KB - 1.00x memory usage +0 KB
add_to_table - 100 chunk size, no_get_tid=false, check_queue_size=false       128.29 KB - 1.28x memory usage +28.10 KB

**All measurements for memory usage were the same**

Reduction count statistics:

Name                                                                    Reduction count
add_to_table - 100 chunk, no_get_tid=true, check_queue_size=false                5.78 K
add_to_table - 100 chunk, no_get_tid=true, check_queue_size=true                 5.78 K - 1.00x reduction count +0 K
add_to_table - 100 chunk size, no_get_tid=false, check_queue_size=false          6.99 K - 1.21x reduction count +1.21 K

@Ziinc Ziinc changed the base branch from main to perf/increase-chunk-size-add-to-table January 5, 2026 16:07
@Ziinc Ziinc force-pushed the perf/remove-get-tid-usage branch from 6feb8c8 to d946d7b Compare January 5, 2026 16:08
Base automatically changed from perf/increase-chunk-size-add-to-table to main January 5, 2026 16:19
@Ziinc Ziinc force-pushed the perf/remove-get-tid-usage branch from d946d7b to 67b6658 Compare January 5, 2026 16:22
@Ziinc Ziinc merged commit 8b9b86f into main Jan 5, 2026
7 checks passed
@Ziinc Ziinc deleted the perf/remove-get-tid-usage branch January 5, 2026 16:53
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