Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compressing timestamps #9

Open
fatihky opened this issue Mar 3, 2017 · 6 comments
Open

Compressing timestamps #9

fatihky opened this issue Mar 3, 2017 · 6 comments

Comments

@fatihky
Copy link

fatihky commented Mar 3, 2017

I think compressing timestamps before writing to disk would be cool. Even when storing on ram. I don't know any compression libraries for doubles(data type that used for values), so I can't suggest anything to compress values.

How about this idea?

Example integer compression libraries:
https://github.com/lemire/simdcomp (BSD)
https://github.com/powturbo/TurboPFor (GPL V2)

@danni-m
Copy link
Owner

danni-m commented Mar 3, 2017

This is a great idea, the links seem interesting.
I was thinking of implementing something similar to beringei encoding algorithm.
The issue i see with these libraries is that they act on complete array, I would prefer to have something that looks more like a streaming interface, every read/write operation will be done ontop of the compressed data and not as a bulk.

Also, antirez is working on a timestamp based streaming data stracture that will have some compression, I might use that in the future.

@fatihky
Copy link
Author

fatihky commented Mar 3, 2017

Cool, then I am closing this.
Is antirez's work this?

@fatihky fatihky closed this as completed Mar 3, 2017
@fatihky
Copy link
Author

fatihky commented Mar 3, 2017

Sorry I didn't ask, is closing this ok?

@danni-m
Copy link
Owner

danni-m commented Mar 3, 2017

@fatihky regarding RCP11, yes.
I'll reopen, I believe that this is an important feature, maybe in the future i'll break it down to 2 different features:

  1. on disk compression
  2. in memory compression

Thanks for the feedback!

@danni-m danni-m reopened this Mar 3, 2017
@fatihky
Copy link
Author

fatihky commented Mar 3, 2017

Thank you for information, and you are welcome.

@danni-m danni-m moved this from Bugs to Work In Progress in redis-module-timeseries Mar 8, 2017
@danni-m danni-m moved this from Work In Progress to Future Tasks in redis-module-timeseries Mar 8, 2017
@danni-m danni-m closed this as completed Mar 11, 2017
@danni-m danni-m reopened this Mar 11, 2017
@danni-m
Copy link
Owner

danni-m commented Mar 11, 2017

Sorry for the mess, i've mistaken this ticket with the rollups/compcations.

@danni-m danni-m moved this from Future Tasks to Work In Progress in redis-module-timeseries Apr 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
redis-module-timeseries
Work In Progress
Development

No branches or pull requests

2 participants