Skip to content

PHOENIX-6450 Checkstyle creating warnings for line length > 80 but < 100 - #1276

Merged
richardantal merged 1 commit into
apache:masterfrom
richardantal:PHOENIX-6450
Jul 29, 2021
Merged

PHOENIX-6450 Checkstyle creating warnings for line length > 80 but < 100#1276
richardantal merged 1 commit into
apache:masterfrom
richardantal:PHOENIX-6450

Conversation

@richardantal

Copy link
Copy Markdown
Contributor

No description provided.

@stoty

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec5m 35sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+1 💚mvninstall21m 27smaster passed
+0hbaserecompile28m 28sHBase recompiled.
+1 💚compile1m 43smaster passed
+1 💚javadoc1m 20smaster passed
_ Patch Compile Tests _
+1 💚mvninstall14m 7sthe patch passed
+0hbaserecompile52m 38sHBase recompiled.
+1 💚compile1m 43sthe patch passed
+1 💚javac1m 43sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 1sThe patch has no ill-formed XML file.
+1 💚javadoc1m 22sthe patch passed
_ Other Tests _
-1 ❌unit117m 59sroot in the patch failed.
-1 ❌asflicense1m 9sThe patch generated 614 ASF License warnings.
183m 23s
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-1276/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1276
Optional Testsdupname asflicense javac javadoc unit xml compile
unameLinux faa997777a64 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 / 6842c1d
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-1276/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-1276/1/testReport/
asflicensehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1276/1/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count13991 (vs. ulimit of 30000)
modulesC: . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1276/1/console
versionsgit=2.7.4 maven=3.3.9
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@richardantal
richardantal requested a review from stotyJuly 28, 2021 10:48
Comment threadpom.xml
<antlr-output.dir>target/generated-sources/antlr3</antlr-output.dir>
<test.output.tofile>true</test.output.tofile>
<top.dir>${project.basedir}</top.dir>
<top.dir>${project.basedir}/..</top.dir>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

top.dir in the submodules doesn't seem to do anything.
I suggest that you delete it from those to simplify the maven setup.

Change-Id: I24ae68bb3c237e1327b2c93774727f328b11aa50

@stotystoty left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

+1 LGTM

@richardantal
richardantal merged commit 693cc1c into apache:masterJul 29, 2021
@stoty

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec4m 24sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+0 🆗mvndep4m 57sMaven dependency ordering for branch
+1 💚mvninstall16m 31smaster passed
+0hbaserecompile28m 29sHBase recompiled.
+1 💚compile1m 44smaster passed
+1 💚javadoc3m 36smaster passed
_ Patch Compile Tests _
+0 🆗mvndep0m 17sMaven dependency ordering for patch
+1 💚mvninstall13m 27sthe patch passed
+0hbaserecompile21m 7sHBase recompiled.
+1 💚compile1m 42sthe patch passed
+1 💚javac1m 42sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 9sThe patch has no ill-formed XML file.
+1 💚javadoc3m 35sthe patch passed
_ Other Tests _
-1 ❌unit116m 48sroot in the patch failed.
-1 ❌asflicense4m 26sThe patch generated 614 ASF License warnings.
189m 49s
ReasonTests
Failed junit testsphoenix.end2end.UpsertSelectIT
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1276/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1276
Optional Testsdupname asflicense javac javadoc unit xml compile
unameLinux c836c3ad11f1 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev/phoenix-personality.sh
git revisionmaster / d04b3d8
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-1276/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-1276/2/testReport/
asflicensehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1276/2/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count15758 (vs. ulimit of 30000)
modulesC: phoenix-core phoenix-pherf phoenix-client-parent phoenix-server phoenix-tracing-webapp phoenix-assembly . U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1276/2/console
versionsgit=2.7.4 maven=3.3.9
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

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

@richardantal@stoty