Skip to content

Conversation

@Apache9
Copy link
Contributor

@Apache9 Apache9 commented Oct 8, 2021

…s failure

@Apache9 Apache9 self-assigned this Oct 8, 2021
@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 35s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 shelldocs 0m 0s Shelldocs was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
+0 🆗 mvndep 0m 18s Maven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗 mvndep 0m 7s Maven dependency ordering for patch
+1 💚 shellcheck 0m 0s There were no new shellcheck issues.
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
_ Other Tests _
+0 🆗 asflicense 0m 0s ASF License check generated no output?
2m 12s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3736/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR #3736
Optional Tests dupname asflicense shellcheck shelldocs
uname Linux f942af955d8c 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 dev-support/hbase-personality.sh
git revision master / 39a20c5
Max. process+thread count 50 (vs. ulimit of 30000)
modules C: U:
Console output https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3736/1/console
versions git=2.17.1 maven=3.6.3 shellcheck=0.4.6
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 0m 59s 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 _
+0 🆗 mvndep 0m 16s Maven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗 mvndep 0m 7s Maven dependency ordering for patch
_ Other Tests _
2m 21s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3736/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR #3736
Optional Tests
uname Linux f548a34b7578 4.15.0-147-generic #151-Ubuntu SMP Fri Jun 18 19:21:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / 39a20c5
Max. process+thread count 50 (vs. ulimit of 30000)
modules C: U:
Console output https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3736/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 14s Docker mode activated.
-0 ⚠️ yetus 0m 2s Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+0 🆗 mvndep 0m 15s Maven dependency ordering for branch
_ Patch Compile Tests _
+0 🆗 mvndep 0m 6s Maven dependency ordering for patch
_ Other Tests _
2m 35s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3736/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR #3736
Optional Tests
uname Linux a99e1ea80ec9 4.15.0-147-generic #151-Ubuntu SMP Fri Jun 18 19:21:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / 39a20c5
Max. process+thread count 45 (vs. ulimit of 30000)
modules C: U:
Console output https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3736/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 Author

Apache9 commented Oct 8, 2021

This is the pre commit result
#3733

And for nightlies build
https://ci-hadoop.apache.org/job/HBase/job/HBase%20Nightly/job/HBASE-26339/2/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/

There were errors when running junit for JDK11 hadoop3, but we still uploaded the test logs to nightlies
https://nightlies.apache.org/hbase/HBase/HBase%20Nightly/HBASE-26339/2/output-jdk11-hadoop3/

So I think ithe PR could solve the problem describe in the jira issue.

Thanks.

@Apache9 Apache9 requested review from busbey and petersomogyi October 8, 2021 15:13
Copy link
Member

@joshelser joshelser left a comment

Choose a reason for hiding this comment

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

Makes me wish that script had a built-in callback you could give it if the script returned non-zero, but alas.

LGTM

Copy link
Member

@ndimiduk ndimiduk left a comment

Choose a reason for hiding this comment

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

If it works, yeah, let's use it. I wonder though, when is post { always { ... } } supposed to run, if not "always" ?

Copy link
Contributor

@busbey busbey left a comment

Choose a reason for hiding this comment

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

oof. thanks for including the comment about why we're doing this.

@Apache9 Apache9 merged commit c410bae into master Oct 9, 2021
Apache9 added a commit that referenced this pull request Oct 9, 2021
…s failure (#3736)

Signed-off-by: Josh Elser <[email protected]>
Signed-off-by: Nick Dimiduk <[email protected]>
Signed-off-by: Sean Busbey <[email protected]>
Apache9 added a commit that referenced this pull request Oct 9, 2021
…s failure (#3736)

Signed-off-by: Josh Elser <[email protected]>
Signed-off-by: Nick Dimiduk <[email protected]>
Signed-off-by: Sean Busbey <[email protected]>
Apache9 added a commit that referenced this pull request Oct 9, 2021
…s failure (#3736)

Signed-off-by: Josh Elser <[email protected]>
Signed-off-by: Nick Dimiduk <[email protected]>
Signed-off-by: Sean Busbey <[email protected]>
Apache9 added a commit that referenced this pull request Oct 9, 2021
…s failure (#3736)

Signed-off-by: Josh Elser <[email protected]>
Signed-off-by: Nick Dimiduk <[email protected]>
Signed-off-by: Sean Busbey <[email protected]>
Apache9 added a commit that referenced this pull request Oct 9, 2021
…s failure (#3736)

Signed-off-by: Josh Elser <[email protected]>
Signed-off-by: Nick Dimiduk <[email protected]>
Signed-off-by: Sean Busbey <[email protected]>
@Apache9 Apache9 deleted the HBASE-26339 branch October 9, 2021 02:14
udaynpusa pushed a commit to mapr/hbase that referenced this pull request Jan 30, 2024
…s failure (apache#3736)

Signed-off-by: Josh Elser <[email protected]>
Signed-off-by: Nick Dimiduk <[email protected]>
Signed-off-by: Sean Busbey <[email protected]>
vinayakphegde pushed a commit to vinayakphegde/hbase that referenced this pull request Apr 4, 2024
…s failure (apache#3736)

Signed-off-by: Josh Elser <[email protected]>
Signed-off-by: Nick Dimiduk <[email protected]>
Signed-off-by: Sean Busbey <[email protected]>
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.

5 participants