Skip to content

test reuseFork=false impact on QA results and perf trade-off - #1080

Closed
virajjasani wants to merge 3 commits into
apache:masterfrom
virajjasani:reuseFork
Closed

test reuseFork=false impact on QA results and perf trade-off#1080
virajjasani wants to merge 3 commits into
apache:masterfrom
virajjasani:reuseFork

Conversation

@virajjasani

@virajjasanivirajjasani commented Jan 9, 2021•
edited
Loading

Copy link
Copy Markdown
Contributor

Part of PHOENIX-6288

@stoty

stoty commented Jan 9, 2021

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 7sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 1 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep5m 40sMaven dependency ordering for branch
+1 💚mvninstall11m 55smaster passed
+1 💚compile1m 27smaster passed
+1 💚checkstyle0m 33smaster passed
+1 💚javadoc2m 1smaster passed
+0 🆗spotbugs3m 9sphoenix-core in master has 956 extant spotbugs warnings.
+0 🆗spotbugs4m 16sroot in master has 1005 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 14sMaven dependency ordering for patch
+1 💚mvninstall9m 18sthe patch passed
+1 💚compile1m 27sthe patch passed
+1 💚javac1m 27sthe patch passed
+1 💚checkstyle0m 32sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 1sThe patch has no ill-formed XML file.
+1 💚javadoc2m 2sthe patch passed
+1 💚spotbugs7m 54sthe patch passed
_ Other Tests _
-1 ❌unit233m 53sroot in the patch failed.
+1 💚asflicense1m 12sThe patch does not generate ASF License warnings.
289m 32s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1080
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile xml
unameLinux b318dc07b077 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / 9c346ed
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/1/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/1/testReport/
Max. process+thread count5772 (vs. ulimit of 30000)
modulesC: phoenix-core . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/1/console
versionsgit=2.7.4 maven=3.3.9 spotbugs=4.1.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@stoty

stoty commented Jan 10, 2021

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 8sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 1 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep5m 14sMaven dependency ordering for branch
+1 💚mvninstall11m 39smaster passed
+1 💚compile1m 26smaster passed
+1 💚checkstyle0m 33smaster passed
+1 💚javadoc2m 3smaster passed
+0 🆗spotbugs3m 11sphoenix-core in master has 956 extant spotbugs warnings.
+0 🆗spotbugs4m 22sroot in master has 1005 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for patch
+1 💚mvninstall9m 14sthe patch passed
+1 💚compile1m 28sthe patch passed
+1 💚javac1m 28sthe patch passed
+1 💚checkstyle0m 33sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 1sThe patch has no ill-formed XML file.
+1 💚javadoc2m 2sthe patch passed
+1 💚spotbugs7m 51sthe patch passed
_ Other Tests _
-1 ❌unit275m 38sroot in the patch failed.
+1 💚asflicense0m 18sThe patch does not generate ASF License warnings.
328m 13s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1080
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile xml
unameLinux bff9f47fdc61 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / 9c346ed
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/2/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/2/testReport/
Max. process+thread count5697 (vs. ulimit of 30000)
modulesC: phoenix-core . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/2/console
versionsgit=2.7.4 maven=3.3.9 spotbugs=4.1.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@stoty

stoty commented Jan 10, 2021

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 18sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 1 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep5m 6sMaven dependency ordering for branch
+1 💚mvninstall11m 45smaster passed
+1 💚compile1m 26smaster passed
+1 💚checkstyle0m 33smaster passed
+1 💚javadoc2m 3smaster passed
+0 🆗spotbugs3m 11sphoenix-core in master has 956 extant spotbugs warnings.
+0 🆗spotbugs4m 18sroot in master has 1005 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for patch
+1 💚mvninstall9m 20sthe patch passed
+1 💚compile1m 27sthe patch passed
+1 💚javac1m 27sthe patch passed
+1 💚checkstyle0m 33sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 1sThe patch has no ill-formed XML file.
+1 💚javadoc2m 4sthe patch passed
+1 💚spotbugs7m 53sthe patch passed
_ Other Tests _
-1 ❌unit233m 55sroot in the patch failed.
+1 💚asflicense0m 19sThe patch does not generate ASF License warnings.
286m 46s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/3/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1080
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile xml
unameLinux ffe0865e6c2d 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / 9c346ed
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/3/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/3/testReport/
Max. process+thread count5710 (vs. ulimit of 30000)
modulesC: phoenix-core . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/3/console
versionsgit=2.7.4 maven=3.3.9 spotbugs=4.1.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@stoty

stoty commented Jan 10, 2021

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 13sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 1 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep5m 3sMaven dependency ordering for branch
+1 💚mvninstall11m 35smaster passed
+1 💚compile1m 27smaster passed
+1 💚checkstyle0m 33smaster passed
+1 💚javadoc2m 2smaster passed
+0 🆗spotbugs3m 9sphoenix-core in master has 955 extant spotbugs warnings.
+0 🆗spotbugs4m 19sroot in master has 1004 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 15sMaven dependency ordering for patch
+1 💚mvninstall9m 8sthe patch passed
+1 💚compile1m 27sthe patch passed
+1 💚javac1m 27sthe patch passed
+1 💚checkstyle0m 31sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 2sThe patch has no ill-formed XML file.
+1 💚javadoc2m 5sthe patch passed
+1 💚spotbugs7m 53sthe patch passed
_ Other Tests _
-1 ❌unit236m 25sroot in the patch failed.
+1 💚asflicense0m 19sThe patch does not generate ASF License warnings.
288m 45s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/4/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1080
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile xml
unameLinux a97a17d7a567 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / fe0b3ca
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/4/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/4/testReport/
Max. process+thread count5723 (vs. ulimit of 30000)
modulesC: phoenix-core . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/4/console
versionsgit=2.7.4 maven=3.3.9 spotbugs=4.1.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@stoty

stoty commented Jan 13, 2021

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 56sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 1 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep5m 5sMaven dependency ordering for branch
+1 💚mvninstall11m 50smaster passed
+1 💚compile1m 41smaster passed
+1 💚checkstyle0m 37smaster passed
+1 💚javadoc2m 12smaster passed
+0 🆗spotbugs3m 31sphoenix-core in master has 955 extant spotbugs warnings.
+0 🆗spotbugs4m 51sroot in master has 1004 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 17sMaven dependency ordering for patch
+1 💚mvninstall9m 20sthe patch passed
+1 💚compile1m 42sthe patch passed
+1 💚javac1m 42sthe patch passed
+1 💚checkstyle0m 37sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 2sThe patch has no ill-formed XML file.
+1 💚javadoc2m 4sthe patch passed
+1 💚spotbugs8m 30sthe patch passed
_ Other Tests _
-1 ❌unit179m 31sroot in the patch failed.
+1 💚asflicense0m 20sThe patch does not generate ASF License warnings.
234m 34s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/5/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1080
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile xml
unameLinux 6dc91e83a181 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 toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / 6d43aa2
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/5/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/5/testReport/
Max. process+thread count6213 (vs. ulimit of 30000)
modulesC: phoenix-core . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/5/console
versionsgit=2.7.4 maven=3.3.9 spotbugs=4.1.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@stoty

stoty commented Jan 17, 2021

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 13sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 1 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep4m 52sMaven dependency ordering for branch
+1 💚mvninstall12m 54smaster passed
+0hbaserecompile25m 49sHBase recompiled.
+1 💚compile1m 36smaster passed
+1 💚checkstyle0m 38smaster passed
+1 💚javadoc2m 12smaster passed
+0 🆗spotbugs4m 46sroot in master has 1004 extant spotbugs warnings.
+0 🆗spotbugs2m 59sphoenix-core in master has 955 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 17sMaven dependency ordering for patch
+1 💚mvninstall10m 3sthe patch passed
+0hbaserecompile18m 10sHBase recompiled.
+1 💚compile1m 36sthe patch passed
+1 💚javac1m 36sthe patch passed
+1 💚checkstyle0m 38sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 1sThe patch has no ill-formed XML file.
+1 💚javadoc2m 10sthe patch passed
+1 💚spotbugs8m 10sthe patch passed
_ Other Tests _
-1 ❌unit167m 58sroot in the patch failed.
-1 ❌asflicense0m 39sThe patch generated 610 ASF License warnings.
240m 27s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/6/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1080
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile xml
unameLinux 9345c6a4d8ea 4.15.0-128-generic #131-Ubuntu SMP Wed Dec 9 06:57:35 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / f80e221
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/6/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/6/testReport/
asflicensehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/6/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count9348 (vs. ulimit of 30000)
modulesC: phoenix-core . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/6/console
versionsgit=2.7.4 maven=3.3.9 spotbugs=4.1.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@stoty

stoty commented Jan 20, 2021

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 29sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 1 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep4m 50sMaven dependency ordering for branch
+1 💚mvninstall11m 18smaster passed
+0hbaserecompile22m 40sHBase recompiled.
+1 💚compile1m 33smaster passed
+1 💚checkstyle0m 38smaster passed
+1 💚javadoc2m 11smaster passed
+0 🆗spotbugs4m 23sroot in master has 1004 extant spotbugs warnings.
+0 🆗spotbugs2m 47sphoenix-core in master has 955 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 17sMaven dependency ordering for patch
+1 💚mvninstall8m 12sthe patch passed
+0hbaserecompile15m 0sHBase recompiled.
+1 💚compile1m 33sthe patch passed
+1 💚javac1m 33sthe patch passed
+1 💚checkstyle0m 39sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 1sThe patch has no ill-formed XML file.
+1 💚javadoc2m 11sthe patch passed
+1 💚spotbugs7m 33sthe patch passed
_ Other Tests _
-1 ❌unit137m 28sroot in the patch failed.
-1 ❌asflicense2m 26sThe patch generated 610 ASF License warnings.
205m 4s
ReasonTests
Failed junit testsphoenix.end2end.PermissionNSEnabledWithCustomAccessControllerIT
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/7/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1080
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile xml
unameLinux b1c170a7d4ea 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 toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / ded6d73
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/7/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/7/testReport/
asflicensehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/7/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count13293 (vs. ulimit of 30000)
modulesC: phoenix-core . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/7/console
versionsgit=2.7.4 maven=3.3.9 spotbugs=4.1.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@stoty

stoty commented Jan 20, 2021

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 26sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 1 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep4m 32sMaven dependency ordering for branch
+1 💚mvninstall11m 10smaster passed
+0hbaserecompile21m 54sHBase recompiled.
+1 💚compile1m 30smaster passed
+1 💚checkstyle0m 38smaster passed
+1 💚javadoc2m 15smaster passed
+0 🆗spotbugs4m 18sroot in master has 1004 extant spotbugs warnings.
+0 🆗spotbugs2m 49sphoenix-core in master has 955 extant spotbugs warnings.
-0 ⚠️patch3m 4sUsed diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for patch
+1 💚mvninstall8m 12sthe patch passed
+0hbaserecompile14m 43sHBase recompiled.
+1 💚compile1m 30sthe patch passed
+1 💚javac1m 30sthe patch passed
+1 💚checkstyle0m 39sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 1sThe patch has no ill-formed XML file.
+1 💚javadoc2m 6sthe patch passed
+1 💚spotbugs8m 2sthe patch passed
_ Other Tests _
-1 ❌unit141m 11sroot in the patch failed.
-1 ❌asflicense2m 13sThe patch generated 610 ASF License warnings.
207m 54s
ReasonTests
Failed junit testsphoenix.end2end.PermissionNSEnabledWithCustomAccessControllerIT
phoenix.end2end.PermissionNSEnabledIT
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/8/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1080
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile xml
unameLinux 44cd4729bd24 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 toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / 772ff2c
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/8/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/8/testReport/
asflicensehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/8/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count12692 (vs. ulimit of 30000)
modulesC: phoenix-core . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/8/console
versionsgit=2.7.4 maven=3.3.9 spotbugs=4.1.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@virajjasanivirajjasani mentioned this pull request Jan 22, 2021
Closed
@stoty

stoty commented Jan 22, 2021

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 34sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚test4tests0m 0sThe patch appears to include 1 new or modified test files.
_ master Compile Tests _
+0 🆗mvndep4m 40sMaven dependency ordering for branch
+1 💚mvninstall14m 43smaster passed
+0hbaserecompile27m 39sHBase recompiled.
+1 💚compile1m 52smaster passed
+1 💚checkstyle0m 46smaster passed
+1 💚javadoc2m 36smaster passed
+0 🆗spotbugs5m 38sroot in master has 1004 extant spotbugs warnings.
+0 🆗spotbugs3m 35sphoenix-core in master has 955 extant spotbugs warnings.
-0 ⚠️patch3m 53sUsed diff version of patch file. Binary files and potentially other changes not applied. Please rebase and squash commits if necessary.
_ Patch Compile Tests _
+0 🆗mvndep0m 19sMaven dependency ordering for patch
+1 💚mvninstall11m 36sthe patch passed
+0hbaserecompile19m 53sHBase recompiled.
+1 💚compile1m 31sthe patch passed
+1 💚javac1m 31sthe patch passed
+1 💚checkstyle0m 37sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 1sThe patch has no ill-formed XML file.
+1 💚javadoc2m 11sthe patch passed
+1 💚spotbugs7m 42sthe patch passed
_ Other Tests _
-1 ❌unit138m 36sroot in the patch failed.
-1 ❌asflicense2m 33sThe patch generated 610 ASF License warnings.
219m 29s
ReasonTests
Failed junit testsphoenix.end2end.PermissionNSEnabledWithCustomAccessControllerIT
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/9/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1080
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile xml
unameLinux fd84c55c55ff 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / 11b37e6
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/9/artifact/yetus-general-check/output/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/9/testReport/
asflicensehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/9/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count13309 (vs. ulimit of 30000)
modulesC: phoenix-core . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1080/9/console
versionsgit=2.7.4 maven=3.3.9 spotbugs=4.1.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@virajjasani

virajjasani commented Jan 22, 2021

Copy link
Copy Markdown
ContributorAuthor

Closing this as #1101 is in.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@virajjasani@stoty