-
Notifications
You must be signed in to change notification settings - Fork 28k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[SPARK-47310][SS] Add micro-benchmark for merge operations for multip…
…le values in value portion of state store ### What changes were proposed in this pull request? Add microbenchmark for merge operations for multiple values in value portion of state store ### Why are the changes needed? Micro-benchmark to understand performance with/without rows tracking around merge operations As shown in the results, merge without tracking is consistently 3x faster ``` merging 10000 rows with 10 values per key (10000 rows to overwrite - rate 100): Best Time(ms) Avg Time(ms) Stdev(ms) Rate(M/s) Per Row(ns) Relative -------------------------------------------------------------------------------------------------------------------------------------------------------------- RocksDB (trackTotalNumberOfRows: true) 519 533 7 0.0 51916.6 1.0X RocksDB (trackTotalNumberOfRows: false) 171 177 3 0.1 17083.9 3.0X ``` GH Actions here: - https://github.com/anishshri-db/spark/actions/runs/8559698160 - https://github.com/anishshri-db/spark/actions/runs/8559694994 Difference is even more running locally (> 7x faster without tracking) ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Test only change ### Was this patch authored or co-authored using generative AI tooling? No Closes #45865 from anishshri-db/task/SPARK-47310. Authored-by: Anish Shrigondekar <anish.shrigondekar@databricks.com> Signed-off-by: Jungtaek Lim <kabhwan.opensource@gmail.com>
- Loading branch information
1 parent
18072b5
commit 1efbf43
Showing
3 changed files
with
265 additions
and
79 deletions.
There are no files selected for viewing
This file contains 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
Oops, something went wrong.