You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug, including details regarding any error messages, version, and platform.
I've found a typo in the build_orc macro in the ThirdPartyToolchain, which means that the orc build command defaults to downloading zstd, instead of using the same configuration as the main arrow build command.
…34164)
This change fixes a variable name and specifies more Zstandard related parameters explicitly to use the same Zstandard used in Apache Arrow itself.
* Closes: #34163
Lead-authored-by: NoahFournier <63198198+NoahFournier@users.noreply.github.com>
Co-authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
…d ORC (apache#34164)
This change fixes a variable name and specifies more Zstandard related parameters explicitly to use the same Zstandard used in Apache Arrow itself.
* Closes: apache#34163
Lead-authored-by: NoahFournier <63198198+NoahFournier@users.noreply.github.com>
Co-authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
fatemehp
pushed a commit
to fatemehp/arrow
that referenced
this issue
Feb 24, 2023
…d ORC (apache#34164)
This change fixes a variable name and specifies more Zstandard related parameters explicitly to use the same Zstandard used in Apache Arrow itself.
* Closes: apache#34163
Lead-authored-by: NoahFournier <63198198+NoahFournier@users.noreply.github.com>
Co-authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Describe the bug, including details regarding any error messages, version, and platform.
I've found a typo in the build_orc macro in the ThirdPartyToolchain, which means that the orc build command defaults to downloading zstd, instead of using the same configuration as the main arrow build command.
"-DZSTD_HOME=${ORZ_ZSTD_ROOT}")
(From here)Component(s)
C++, Continuous Integration
The text was updated successfully, but these errors were encountered: