Display the total Growth of all CF-s (#17)

speedb-io / log-parser

A tool for analyzing and parsing SpeedB and RocksDB log files

Apache License 2.0

21 stars 1 forks source link

Display the total Growth of all CF-s (#17) #39

Closed udi-speedb closed 1 year ago

Yuval-Ariel commented 1 year ago

seems a bit off. says the total growth is negative but each sum in positive. maybe its the unknown cfs? i believe we should exclude them from the math somehow. maybe only report the cfs that have both data (before and after) and specify for how many cfs the total growth is for.

LOG-readrandomwriterandom_50.gz

udi-speedb commented 1 year ago

seems a bit off. says the total growth is negative but each sum in positive. maybe its the unknown cfs? i believe we should exclude them from the math somehow. maybe only report the cfs that have both data (before and after) and specify for how many cfs the total growth is for.

LOG-readrandomwriterandom_50.gz

You are correct. I am implementing you suggestions and will submit an updated branch when done.

udi-speedb commented 1 year ago

@Yuval-Ariel - Please check now. Thanks

Yuval-Ariel commented 1 year ago

looks great!