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

[C++] Dependency: Upgrade zstd to v1.5.5 #34899

Closed
mapleFU opened this issue Apr 5, 2023 · 0 comments · Fixed by #34900
Closed

[C++] Dependency: Upgrade zstd to v1.5.5 #34899

mapleFU opened this issue Apr 5, 2023 · 0 comments · Fixed by #34900

Comments

@mapleFU
Copy link
Member

mapleFU commented Apr 5, 2023

Describe the enhancement requested

Currently, apache-arrow using zstd v1.5.2, previously, I try to upgrade it to v1.5.4, however, a CMake problem in zstd v1.5.4 prevents that.

Now zstd has a v1.5.5 release, I'd like to try it

Component(s)

C++

@kou kou closed this as completed in #34900 Apr 5, 2023
kou pushed a commit that referenced this issue Apr 5, 2023
### Rationale for this change

Upgrade zstd from v1.5.2 to v.1.5.5

### What changes are included in this PR?

zstd version upgrade

### Are these changes tested?

No, waiting for ut

### Are there any user-facing changes?

No

* Closes: #34899

Authored-by: mwish <maplewish117@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
@kou kou added this to the 12.0.0 milestone Apr 5, 2023
ArgusLi pushed a commit to Bit-Quill/arrow that referenced this issue May 15, 2023
### Rationale for this change

Upgrade zstd from v1.5.2 to v.1.5.5

### What changes are included in this PR?

zstd version upgrade

### Are these changes tested?

No, waiting for ut

### Are there any user-facing changes?

No

* Closes: apache#34899

Authored-by: mwish <maplewish117@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
rtpsw pushed a commit to rtpsw/arrow that referenced this issue May 16, 2023
### Rationale for this change

Upgrade zstd from v1.5.2 to v.1.5.5

### What changes are included in this PR?

zstd version upgrade

### Are these changes tested?

No, waiting for ut

### Are there any user-facing changes?

No

* Closes: apache#34899

Authored-by: mwish <maplewish117@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants