Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/22160 )

Change subject: [thirdparty] upgrade LZ4 up to 1.10.0 version
......................................................................

[thirdparty] upgrade LZ4 up to 1.10.0 version

As per LZ4 1.10.0 release notes [1], it makes sense to upgrade LZ4
from 1.9.4 to 1.10.0 version.  At least, the summary mentions that
upgrading to 1.10.0 version is a good step for users looking to
optimize performance in high-throughput environments:

  LZ4 v1.10.0 introduces major updates, integrating 600+ commits that
  significantly enhance its capabilities. This version brings
  multithreading support to the forefront, harnessing modern multi-core
  processors to accelerate both compression and decompression
  processing. It's a good upgrade for users looking to optimize
  performance in high-throughput environments.

As a quick verification, I built and ran the compression-test
(RELEASE bits) for 1.9.4 and 1.10.0 LZ4 at the same machine.
The results show performance improvement for 1.10.0 compared with
1.9.4 LZ4 version:

  LZ4 1.9.4:
    LZ4 compress throughput: 718.036 MB/sec, ratio: 0.358154
    LZ4 uncompress throughput: 2928.19 MB/sec

  LZ4 1.10.0:
    LZ4 compress throughput: 735.147 MB/sec, ratio: 0.360474
    LZ4 uncompress throughput: 2942.13 MB/sec

[1] https://github.com/lz4/lz4/releases/tag/v1.10.0

Change-Id: Ie1eff7e2977db007436ec82995cf30f3fd5472ca
Reviewed-on: http://gerrit.cloudera.org:8080/22160
Reviewed-by: Mahesh Reddy <[email protected]>
Tested-by: Alexey Serbin <[email protected]>
Reviewed-by: Abhishek Chennaka <[email protected]>
---
M thirdparty/vars.sh
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Mahesh Reddy: Looks good to me, but someone else must approve
  Alexey Serbin: Verified
  Abhishek Chennaka: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/22160
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie1eff7e2977db007436ec82995cf30f3fd5472ca
Gerrit-Change-Number: 22160
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Yifan Zhang <[email protected]>

Reply via email to