+1
On Fri, Feb 15, 2019 at 12:14 PM Dikang Gu wrote:
> +1
>
> On Fri, Feb 15, 2019 at 10:27 AM Vinay Chella
> wrote:
>
> > We have been using Zstd compressor across different products/services
> here
> > and have seen significant improvements, getting this in 4.0 would be a
> big
> > win.
> >
+1
On Fri, Feb 15, 2019 at 10:27 AM Vinay Chella
wrote:
> We have been using Zstd compressor across different products/services here
> and have seen significant improvements, getting this in 4.0 would be a big
> win.
>
> +1
>
> Thanks,
> Vinay Chella
>
>
> On Fri, Feb 15, 2019 at 10:19 AM Jeff
We have been using Zstd compressor across different products/services here
and have seen significant improvements, getting this in 4.0 would be a big
win.
+1
Thanks,
Vinay Chella
On Fri, Feb 15, 2019 at 10:19 AM Jeff Jirsa wrote:
> +1
>
> --
> Jeff Jirsa
>
>
> > On Feb 15, 2019, at 9:35 AM,
Thanks Ariel & Jonathan.
Michael, I have addressed the comments in the issue so we're using the latest
jar now. I've posted the updated patch in the latest comment of the ticket. In
addition there is a JMH performance benchmark in the C* repo
(CompressorPerformance) that tests various
Seems low risk, potentially high reward.
I can run some tests next week to get a rough idea of how compression
ratios differ as well as if there's a difference in performance.
I won't be testing correctness, just looking at the performance profile.
Jon
On Fri, Feb 15, 2019 at 9:56 AM Michael
+0.5
I skimmed the jira and github diff and a few things came to mind:
- There are multiple comments about using an older jar than the latest
version.
- I did not see any performance test results to form an opinion on any
gains/caveats as a user. This was the first thing I looked for.
- I did not
Hi,
I am +1 since it's an additional compressor and not the default.
Ariel
On Fri, Feb 15, 2019, at 11:41 AM, Dinesh Joshi wrote:
> Hey folks,
>
> Just wanted to get a pulse on whether we can proceed with ZStd support.
> The consensus on the ticket was that it’s a very valuable addition
>
IMO "add a new compression class that has demonstrable benefits to Sushma
and Joseph" is sufficiently noninvasive that we should allow it into 4.0.
On Fri, Feb 15, 2019 at 10:48 AM Dinesh Joshi
wrote:
> Hey folks,
>
> Just wanted to get a pulse on whether we can proceed with ZStd support.
> The
Hey folks,
Just wanted to get a pulse on whether we can proceed with ZStd support. The
consensus on the ticket was that it’s a very valuable addition without any risk
of destabilizing 4.0. It’s ready to go if there aren’t any objections.
Dinesh
