Skip to content

PHOENIX-6082 : Avoid checkAndPut when altering properties for a table or view with column-encoding enabled - #983

Closed
virajjasani wants to merge 1 commit into
apache:4.xfrom
virajjasani:PHOENIX-6082-4.x
Closed

PHOENIX-6082 : Avoid checkAndPut when altering properties for a table or view with column-encoding enabled#983
virajjasani wants to merge 1 commit into
apache:4.xfrom
virajjasani:PHOENIX-6082-4.x

Conversation

@virajjasani

@virajjasanivirajjasani commented Nov 22, 2020

Copy link
Copy Markdown
Contributor

4.x backport of PR #982

@stoty

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 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.
_ 4.x Compile Tests _
+1 💚mvninstall11m 8s4.x passed
+1 💚compile0m 56s4.x passed
+1 💚checkstyle1m 40s4.x passed
+1 💚javadoc0m 46s4.x passed
+0 🆗spotbugs2m 55sphoenix-core in 4.x has 950 extant spotbugs warnings.
_ Patch Compile Tests _
+1 💚mvninstall5m 20sthe patch passed
+1 💚compile0m 55sthe patch passed
+1 💚javac0m 55sthe patch passed
-1 ❌checkstyle1m 42sphoenix-core: The patch generated 2 new + 2791 unchanged - 5 fixed = 2793 total (was 2796)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 42sthe patch passed
+1 💚spotbugs3m 3sthe patch passed
_ Other Tests _
-1 ❌unit130m 9sphoenix-core in the patch failed.
+1 💚asflicense0m 39sThe patch does not generate ASF License warnings.
163m 8s
SubsystemReport/Notes
DockerClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-983/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#983
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile
unameLinux 0dcc3bedb512 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 revision4.x / e57fcc8
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-983/2/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-983/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-983/2/testReport/
Max. process+thread count6612 (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-983/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 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.
_ 4.x Compile Tests _
+1 💚mvninstall11m 5s4.x passed
+1 💚compile0m 53s4.x passed
+1 💚checkstyle1m 40s4.x passed
+1 💚javadoc0m 46s4.x passed
+0 🆗spotbugs2m 54sphoenix-core in 4.x has 950 extant spotbugs warnings.
_ Patch Compile Tests _
+1 💚mvninstall5m 21sthe patch passed
+1 💚compile0m 55sthe patch passed
+1 💚javac0m 55sthe patch passed
-1 ❌checkstyle1m 41sphoenix-core: The patch generated 2 new + 2791 unchanged - 5 fixed = 2793 total (was 2796)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 46sthe patch passed
+1 💚spotbugs3m 2sthe patch passed
_ Other Tests _
-1 ❌unit137m 14sphoenix-core in the patch failed.
+1 💚asflicense0m 38sThe patch does not generate ASF License warnings.
170m 11s
ReasonTests
Failed junit testsphoenix.end2end.QueryIT
phoenix.end2end.DropIndexedColsIT
phoenix.end2end.index.GlobalMutableNonTxIndexIT
phoenix.end2end.index.DropColumnIT
SubsystemReport/Notes
DockerClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-983/4/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#983
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile
unameLinux b4ebd698d985 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 revision4.x / 7ac4dff
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-983/4/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-983/4/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-983/4/testReport/
Max. process+thread count6455 (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-983/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.

@ChinmaySKulkarniChinmaySKulkarni 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, thanks! Waiting for any other reviews before committing your changes.
Btw, the test failures look unrelated and flaky but I think we should open a Jira to fix these tests/contact INFRA if it is an infra issue (OOM, shutdowns, etc.) WDYT @virajjasani@yanxinyi@gjacoby126 ?

@virajjasani

Copy link
Copy Markdown
ContributorAuthor

Sure @ChinmaySKulkarni , sounds good. Let me try to collect some of these flaky tests results and create Jira accordingly.

@virajjasani
virajjasani deleted the PHOENIX-6082-4.x branch December 7, 2020 07:00
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.

3 participants

@virajjasani@stoty@ChinmaySKulkarni