Skip to content

YETUS-895. Add ZStandard Compression support in Hadoop personality#103

Merged
aajisaka merged 1 commit intoapache:masterfrom
aajisaka:YETUS-895
Mar 31, 2020
Merged

YETUS-895. Add ZStandard Compression support in Hadoop personality#103
aajisaka merged 1 commit intoapache:masterfrom
aajisaka:YETUS-895

Conversation

@aajisaka
Copy link
Member

Copy link
Contributor

@jojochuang jojochuang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see that hadoop's build Dockerfile installs libzstd1-dev at the standard location. https://github.com/apache/hadoop/blob/trunk/dev-support/docker/Dockerfile#L66
So looks like this line alone will work.

@apache-yetus
Copy link

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://builds.apache.org/job/yetus-github-multibranch/job/PR-103/1/console in case of problems.

@apache-yetus
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 48s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
_ Patch Compile Tests _
+1 💚 shellcheck 0m 2s There were no new shellcheck issues.
+1 💚 shelldocs 0m 0s There were no new shelldocs issues.
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
_ Other Tests _
+1 💚 asflicense 0m 10s The patch does not generate ASF License warnings.
2m 16s
Subsystem Report/Notes
Docker ClientAPI=1.40 ServerAPI=1.40 base: https://builds.apache.org/job/yetus-github-multibranch/job/PR-103/1/artifact/out/Dockerfile
GITHUB PR #103
Optional Tests dupname asflicense shellcheck shelldocs
uname Linux ba34e3c5e3a0 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality precommit/src/main/shell/personality/yetus.sh
git revision master / c88ea1a
Max. process+thread count 46 (vs. ulimit of 2000)
modules C: precommit U: precommit
Console output https://builds.apache.org/job/yetus-github-multibranch/job/PR-103/1/console
versions git=2.7.4 maven=3.3.9 shellcheck=0.7.0
Powered by Apache Yetus 0.12.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

@aajisaka
Copy link
Member Author

Confirmed -Drequire.zstd option is added:
apache/hadoop#1927 (comment)

@aajisaka aajisaka merged commit 6e8a02d into apache:master Mar 31, 2020
@aajisaka
Copy link
Member Author

Merged. Thanks @jojochuang for your review.

@aajisaka aajisaka deleted the YETUS-895 branch March 31, 2020 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants