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

HBASE-27292. Fix build failure against Hadoop 3.3.4 due to added dependency on okhttp. #4687

Merged
merged 1 commit into from
Aug 10, 2022

Conversation

iwasakims
Copy link
Member

https://issues.apache.org/jira/browse/HBASE-27292

$ mvn clean install -DskipTests -Phadoop-3.0 -Dhadoop-three.version=3.3.4
...
[ERROR] Found artifact with unexpected contents: '/home/rocky/srcs/hbase/hbase-shaded/hbase-shaded-mapreduce/target/hbase-shaded-mapreduce-3.0.0-alpha-4-SNAPSHOT.jar'
    Please check the following and either correct the build or update
    the allowed list with reasoning.

    okhttp3/
    okhttp3/Request$Builder.class
    ...
    kotlin/
    kotlin/collections/
    kotlin/collections/ArraysUtilJVM.class
    ...

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 3m 4s Docker mode activated.
-0 ⚠️ yetus 0m 3s Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+1 💚 mvninstall 2m 34s master passed
+1 💚 compile 0m 20s master passed
+1 💚 shadedjars 3m 48s branch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 0m 13s master passed
_ Patch Compile Tests _
+1 💚 mvninstall 2m 20s the patch passed
+1 💚 compile 0m 21s the patch passed
+1 💚 javac 0m 21s the patch passed
+1 💚 shadedjars 3m 47s patch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 0m 11s the patch passed
_ Other Tests _
+1 💚 unit 0m 54s hbase-shaded in the patch passed.
18m 48s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4687/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR #4687
Optional Tests javac javadoc unit shadedjars compile
uname Linux d1d1bf870871 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / 5919b30
Default Java AdoptOpenJDK-1.8.0_282-b08
Test Results https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4687/1/testReport/
Max. process+thread count 404 (vs. ulimit of 30000)
modules C: hbase-shaded U: hbase-shaded
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4687/1/console
versions git=2.17.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 2s Docker mode activated.
-0 ⚠️ yetus 0m 3s Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+1 💚 mvninstall 3m 24s master passed
+1 💚 compile 0m 24s master passed
+1 💚 shadedjars 4m 41s branch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 0m 13s master passed
_ Patch Compile Tests _
+1 💚 mvninstall 2m 58s the patch passed
+1 💚 compile 0m 22s the patch passed
+1 💚 javac 0m 22s the patch passed
+1 💚 shadedjars 4m 29s patch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 0m 12s the patch passed
_ Other Tests _
+1 💚 unit 0m 49s hbase-shaded in the patch passed.
19m 37s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4687/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR #4687
Optional Tests javac javadoc unit shadedjars compile
uname Linux 6d5f141be4a2 5.4.0-1081-aws #88~18.04.1-Ubuntu SMP Thu Jun 23 16:29:17 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / 5919b30
Default Java AdoptOpenJDK-11.0.10+9
Test Results https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4687/1/testReport/
Max. process+thread count 415 (vs. ulimit of 30000)
modules C: hbase-shaded U: hbase-shaded
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4687/1/console
versions git=2.17.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 2s 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 _
+1 💚 mvninstall 2m 29s master passed
+1 💚 compile 0m 22s master passed
+1 💚 spotless 0m 43s branch has no errors when running spotless:check.
_ Patch Compile Tests _
+1 💚 mvninstall 2m 6s the patch passed
+1 💚 compile 0m 19s the patch passed
+1 💚 javac 0m 19s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+1 💚 xml 0m 1s The patch has no ill-formed XML file.
+1 💚 hadoopcheck 11m 43s Patch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.
+1 💚 spotless 0m 40s patch has no errors when running spotless:check.
_ Other Tests _
+1 💚 asflicense 0m 9s The patch does not generate ASF License warnings.
24m 46s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4687/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR #4687
Optional Tests dupname asflicense javac hadoopcheck spotless xml compile
uname Linux 07ce176d5198 5.4.0-1025-aws #25~18.04.1-Ubuntu SMP Fri Sep 11 12:03:04 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / 5919b30
Default Java AdoptOpenJDK-1.8.0_282-b08
Max. process+thread count 71 (vs. ulimit of 30000)
modules C: hbase-shaded U: hbase-shaded
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4687/1/console
versions git=2.17.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache9
Copy link
Contributor

Apache9 commented Aug 10, 2022

Verified locally, this PR can fix the build problem when building with 3.3.4.

I think we should add newer hadoop versions to our hadoop check versions.

@Apache9 Apache9 merged commit 0061210 into apache:master Aug 10, 2022
Apache9 pushed a commit that referenced this pull request Aug 10, 2022
…ndency on okhttp. (#4687)

Signed-off-by: Duo Zhang <zhangduo@apache.org>
(cherry picked from commit 0061210)
Apache9 pushed a commit that referenced this pull request Aug 10, 2022
…ndency on okhttp. (#4687)

Signed-off-by: Duo Zhang <zhangduo@apache.org>
(cherry picked from commit 0061210)
Apache9 pushed a commit that referenced this pull request Aug 10, 2022
…ndency on okhttp. (#4687)

Signed-off-by: Duo Zhang <zhangduo@apache.org>
(cherry picked from commit 0061210)
vinayakphegde pushed a commit to vinayakphegde/hbase that referenced this pull request Apr 4, 2024
…ndency on okhttp. (apache#4687)

Signed-off-by: Duo Zhang <zhangduo@apache.org>
(cherry picked from commit 0061210)
(cherry picked from commit c701c12)
Change-Id: I03472f5ee6f71e50e00b759a0d59aa4de95027c1
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