dgryski / go-tsz

Time series compression algorithm from Facebook's Gorilla paper
BSD 2-Clause "Simplified" License
541 stars 66 forks source link

uint64 timestamps #22

Open jmventura opened 6 years ago

jmventura commented 6 years ago

Hi there, are you planning to implement uint64 timestamps as described in the Facebook's paper anytime soon?

Thanks for the great contribution!

tsenart commented 6 years ago

+1

dgryski commented 5 years ago

If you just want uint64 to represent 32-bit epochs, you're probably fine. If you want to move to sub-second resolution with uint64 nanoseconds, then there are more patches required to support larger actual timestamp sizes.

dgryski commented 5 years ago

Also, not sure how to solve this without breaking anybody using this code. Perhaps a tsz (32-bit) and tsz64 types generated from the same base implementation.

tsenart commented 5 years ago

@dgryski: Have you seen my fork?