Skip to content

PHOENIX-6507 - #1262

Closed
comnetwork wants to merge 5 commits into
apache:4.xfrom
comnetwork:4xphoenix6507
Closed

PHOENIX-6507#1262
comnetwork wants to merge 5 commits into
apache:4.xfrom
comnetwork:4xphoenix6507

Conversation

@comnetwork

Copy link
Copy Markdown
Contributor

No description provided.

@stoty

stoty commented Jul 9, 2021

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec4m 44sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 1sNo 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.
_ 4.x Compile Tests _
+1 💚mvninstall18m 29s4.x passed
+1 💚compile1m 4s4.x passed
+1 💚checkstyle0m 28s4.x passed
+1 💚javadoc0m 50s4.x passed
+0 🆗spotbugs3m 13sphoenix-core in 4.x has 957 extant spotbugs warnings.
_ Patch Compile Tests _
+1 💚mvninstall10m 39sthe patch passed
+1 💚compile1m 5sthe patch passed
+1 💚javac1m 5sthe patch passed
-1 ❌checkstyle0m 27sphoenix-core: The patch generated 17 new + 74 unchanged - 24 fixed = 91 total (was 98)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 48sthe patch passed
-1 ❌spotbugs3m 26sphoenix-core generated 2 new + 957 unchanged - 0 fixed = 959 total (was 957)
_ Other Tests _
+1 💚unit143m 16sphoenix-core in the patch passed.
+1 💚asflicense0m 10sThe patch does not generate ASF License warnings.
189m 40s
ReasonTests
FindBugsmodule:phoenix-core
new org.apache.phoenix.expression.KeyValueColumnExpression(byte[], byte[]) may expose internal representation by storing an externally mutable object into KeyValueColumnExpression.cf At KeyValueColumnExpression.java:internal representation by storing an externally mutable object into KeyValueColumnExpression.cf At KeyValueColumnExpression.java:[line 49]
new org.apache.phoenix.expression.KeyValueColumnExpression(byte[], byte[]) may expose internal representation by storing an externally mutable object into KeyValueColumnExpression.cq At KeyValueColumnExpression.java:internal representation by storing an externally mutable object into KeyValueColumnExpression.cq At KeyValueColumnExpression.java:[line 50]
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1262
JIRA IssuePHOENIX-6507
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile
unameLinux 1b4a7361e0c0 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revision4.x / ddd7ff6
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
checkstylehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/1/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt
spotbugshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/1/artifact/yetus-general-check/output/new-spotbugs-phoenix-core.html
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/1/testReport/
Max. process+thread count5885 (vs. ulimit of 30000)
modulesC: phoenix-core U: phoenix-core
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/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

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 30sDocker 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.
_ 4.x Compile Tests _
+1 💚mvninstall17m 55s4.x passed
+1 💚compile1m 1s4.x passed
+1 💚checkstyle0m 28s4.x passed
+1 💚javadoc0m 46s4.x passed
+0 🆗spotbugs3m 2sphoenix-core in 4.x has 957 extant spotbugs warnings.
_ Patch Compile Tests _
-1 ❌mvninstall0m 25sroot in the patch failed.
-1 ❌compile0m 14sphoenix-core in the patch failed.
-1 ❌javac0m 14sphoenix-core in the patch failed.
-1 ❌checkstyle0m 27sphoenix-core: The patch generated 9 new + 72 unchanged - 26 fixed = 81 total (was 98)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 45sthe patch passed
-1 ❌spotbugs0m 13sphoenix-core in the patch failed.
_ Other Tests _
-1 ❌unit0m 12sphoenix-core in the patch failed.
+1 💚asflicense0m 10sThe patch does not generate ASF License warnings.
26m 56s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1262
JIRA IssuePHOENIX-6507
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile
unameLinux 4c010f92e46b 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revision4.x / ddd7ff6
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
mvninstallhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/2/artifact/yetus-general-check/output/patch-mvninstall-root.txt
compilehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/2/artifact/yetus-general-check/output/patch-compile-phoenix-core.txt
javachttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/2/artifact/yetus-general-check/output/patch-compile-phoenix-core.txt
checkstylehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/2/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt
spotbugshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/2/artifact/yetus-general-check/output/patch-spotbugs-phoenix-core.txt
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/2/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/2/testReport/
Max. process+thread count94 (vs. ulimit of 30000)
modulesC: phoenix-core U: phoenix-core
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/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

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 30sDocker 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.
_ 4.x Compile Tests _
+1 💚mvninstall18m 12s4.x passed
+1 💚compile1m 2s4.x passed
+1 💚checkstyle0m 30s4.x passed
+1 💚javadoc0m 48s4.x passed
+0 🆗spotbugs3m 2sphoenix-core in 4.x has 957 extant spotbugs warnings.
_ Patch Compile Tests _
+1 💚mvninstall9m 42sthe patch passed
+1 💚compile1m 1sthe patch passed
+1 💚javac1m 1sthe patch passed
-1 ❌checkstyle0m 28sphoenix-core: The patch generated 9 new + 72 unchanged - 26 fixed = 81 total (was 98)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 46sthe patch passed
-1 ❌spotbugs3m 18sphoenix-core generated 2 new + 957 unchanged - 0 fixed = 959 total (was 957)
_ Other Tests _
-1 ❌unit138m 10sphoenix-core in the patch failed.
+1 💚asflicense0m 51sThe patch does not generate ASF License warnings.
181m 13s
ReasonTests
FindBugsmodule:phoenix-core
new org.apache.phoenix.expression.KeyValueColumnExpression(byte[], byte[]) may expose internal representation by storing an externally mutable object into KeyValueColumnExpression.cf At KeyValueColumnExpression.java:internal representation by storing an externally mutable object into KeyValueColumnExpression.cf At KeyValueColumnExpression.java:[line 50]
new org.apache.phoenix.expression.KeyValueColumnExpression(byte[], byte[]) may expose internal representation by storing an externally mutable object into KeyValueColumnExpression.cq At KeyValueColumnExpression.java:internal representation by storing an externally mutable object into KeyValueColumnExpression.cq At KeyValueColumnExpression.java:[line 51]
Failed junit testsphoenix.end2end.AuditLoggingIT
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/3/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1262
JIRA IssuePHOENIX-6507
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile
unameLinux 5dd0620b1659 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revision4.x / ddd7ff6
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
checkstylehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/3/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt
spotbugshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/3/artifact/yetus-general-check/output/new-spotbugs-phoenix-core.html
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/3/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/3/testReport/
Max. process+thread count6039 (vs. ulimit of 30000)
modulesC: phoenix-core U: phoenix-core
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/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

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.
_ 4.x Compile Tests _
+1 💚mvninstall17m 44s4.x passed
+1 💚compile1m 0s4.x passed
+1 💚checkstyle0m 28s4.x passed
+1 💚javadoc0m 47s4.x passed
+0 🆗spotbugs3m 2sphoenix-core in 4.x has 957 extant spotbugs warnings.
_ Patch Compile Tests _
+1 💚mvninstall9m 35sthe patch passed
+1 💚compile1m 0sthe patch passed
+1 💚javac1m 0sthe patch passed
-1 ❌checkstyle0m 29sphoenix-core: The patch generated 7 new + 72 unchanged - 26 fixed = 79 total (was 98)
+1 💚whitespace0m 1sThe patch has no whitespace issues.
+1 💚javadoc0m 45sthe patch passed
-1 ❌spotbugs3m 18sphoenix-core generated 2 new + 957 unchanged - 0 fixed = 959 total (was 957)
_ Other Tests _
+1 💚unit183m 27sphoenix-core in the patch passed.
+1 💚asflicense0m 11sThe patch does not generate ASF License warnings.
223m 13s
ReasonTests
FindBugsmodule:phoenix-core
new org.apache.phoenix.expression.KeyValueColumnExpression(byte[], byte[]) may expose internal representation by storing an externally mutable object into KeyValueColumnExpression.cf At KeyValueColumnExpression.java:internal representation by storing an externally mutable object into KeyValueColumnExpression.cf At KeyValueColumnExpression.java:[line 50]
new org.apache.phoenix.expression.KeyValueColumnExpression(byte[], byte[]) may expose internal representation by storing an externally mutable object into KeyValueColumnExpression.cq At KeyValueColumnExpression.java:internal representation by storing an externally mutable object into KeyValueColumnExpression.cq At KeyValueColumnExpression.java:[line 51]
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/4/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1262
JIRA IssuePHOENIX-6507
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile
unameLinux 598e78e030d0 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revision4.x / ddd7ff6
Default JavaPrivate Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
checkstylehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/4/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt
spotbugshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/4/artifact/yetus-general-check/output/new-spotbugs-phoenix-core.html
Test Resultshttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/4/testReport/
Max. process+thread count5666 (vs. ulimit of 30000)
modulesC: phoenix-core U: phoenix-core
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/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

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec0m 0sDocker mode activated.
-1 ❌patch0m 5s#1262 does not apply to 4.x. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.
SubsystemReport/Notes
GITHUB PR#1262
JIRA IssuePHOENIX-6507
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1262/5/console
versionsgit=2.17.1
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@stoty

Copy link
Copy Markdown
Contributor

This has been merged a long time ago.

@stotystoty closed this Dec 16, 2022
Sign up for freeto 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.

2 participants

@comnetwork@stoty