-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
GH-35205: [C++][Gandiva] Don't find system Zstandard when we use bundled one #35220
Conversation
|
@github-actions crossbow submit -g verify-rc-source java-jars |
Revision: 10ffcec Submitted crossbow builds: ursacomputing/crossbow @ actions-cc7caa00a4 |
+1 |
…led one (#35220) ### Rationale for this change If we use bundled Zstandared, we should not find system Zstandard for Gandiva. Because it's always failed. ### What changes are included in this PR? Add a missing check whether we're using system Zstandard or bundled Zstandard. ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * Closes: #35205 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Benchmark runs are scheduled for baseline = d7c5e4c and contender = ce60f2f. ce60f2f is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
['Python', 'R'] benchmarks have high level of regressions. |
…e bundled one (apache#35220) ### Rationale for this change If we use bundled Zstandared, we should not find system Zstandard for Gandiva. Because it's always failed. ### What changes are included in this PR? Add a missing check whether we're using system Zstandard or bundled Zstandard. ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * Closes: apache#35205 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
…e bundled one (apache#35220) ### Rationale for this change If we use bundled Zstandared, we should not find system Zstandard for Gandiva. Because it's always failed. ### What changes are included in this PR? Add a missing check whether we're using system Zstandard or bundled Zstandard. ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * Closes: apache#35205 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
…e bundled one (apache#35220) ### Rationale for this change If we use bundled Zstandared, we should not find system Zstandard for Gandiva. Because it's always failed. ### What changes are included in this PR? Add a missing check whether we're using system Zstandard or bundled Zstandard. ### Are these changes tested? Yes. ### Are there any user-facing changes? Yes. * Closes: apache#35205 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Rationale for this change
If we use bundled Zstandared, we should not find system Zstandard for Gandiva.
Because it's always failed.
What changes are included in this PR?
Add a missing check whether we're using system Zstandard or bundled Zstandard.
Are these changes tested?
Yes.
Are there any user-facing changes?
Yes.