Skip to content

PHOENIX-6444 Extend Cell Tags to Delete object for Indexer coproc - #1197

Merged
gjacoby126 merged 5 commits into
apache:masterfrom
shahrs87:PHOENIX-6444
Apr 21, 2021
Merged

PHOENIX-6444 Extend Cell Tags to Delete object for Indexer coproc#1197
gjacoby126 merged 5 commits into
apache:masterfrom
shahrs87:PHOENIX-6444

Conversation

@shahrs87

@shahrs87shahrs87 commented Apr 13, 2021

Copy link
Copy Markdown
Contributor

No description provided.

@stoty

stoty commented Apr 13, 2021

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 9sDocker 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.
_ master Compile Tests _
-1 ❌mvninstall6m 4sroot in master failed.
+0hbaserecompile13m 5sHBase recompiled.
+1 💚compile1m 3smaster passed
+1 💚checkstyle0m 49smaster passed
+1 💚javadoc0m 53smaster passed
+0 🆗spotbugs3m 20sphoenix-core in master has 960 extant spotbugs warnings.
_ Patch Compile Tests _
-1 ❌mvninstall1m 17sroot in the patch failed.
+0hbaserecompile12m 10sHBase recompiled.
+1 💚compile1m 5sthe patch passed
+1 💚javac1m 5sthe patch passed
-1 ❌checkstyle0m 49sphoenix-core: The patch generated 25 new + 1115 unchanged - 18 fixed = 1140 total (was 1133)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 52sthe patch passed
+1 💚spotbugs3m 31sthe patch passed
_ Other Tests _
-1 ❌unit115m 45sphoenix-core in the patch failed.
+1 💚asflicense0m 35sThe patch does not generate ASF License warnings.
154m 44s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1197
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile
unameLinux 022c12ba7eb6 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 / 958ec62
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-1197/1/artifact/yetus-general-check/output/branch-mvninstall-root.txt
mvninstallhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/1/artifact/yetus-general-check/output/patch-mvninstall-root.txt
checkstylehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/1/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/1/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-1197/1/testReport/
Max. process+thread count10389 (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-1197/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 Apr 14, 2021

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 10sDocker 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.
_ master Compile Tests _
-1 ❌mvninstall6m 1sroot in master failed.
+0hbaserecompile13m 1sHBase recompiled.
+1 💚compile1m 4smaster passed
+1 💚checkstyle0m 48smaster passed
+1 💚javadoc0m 52smaster passed
+0 🆗spotbugs3m 21sphoenix-core in master has 960 extant spotbugs warnings.
_ Patch Compile Tests _
-1 ❌mvninstall1m 17sroot in the patch failed.
+0hbaserecompile11m 53sHBase recompiled.
+1 💚compile1m 4sthe patch passed
+1 💚javac1m 4sthe patch passed
-1 ❌checkstyle0m 50sphoenix-core: The patch generated 18 new + 1122 unchanged - 11 fixed = 1140 total (was 1133)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 53sthe patch passed
+1 💚spotbugs3m 29sthe patch passed
_ Other Tests _
-1 ❌unit117m 23sphoenix-core in the patch failed.
+1 💚asflicense0m 34sThe patch does not generate ASF License warnings.
156m 1s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1197
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile
unameLinux 7f0734794b4b 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 / 958ec62
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-1197/2/artifact/yetus-general-check/output/branch-mvninstall-root.txt
mvninstallhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/2/artifact/yetus-general-check/output/patch-mvninstall-root.txt
checkstylehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/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-1197/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-1197/2/testReport/
Max. process+thread count10877 (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-1197/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 Apr 17, 2021

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 6sDocker 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 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 💚mvninstall17m 2smaster passed
+0hbaserecompile24m 1sHBase recompiled.
+1 💚compile1m 3smaster passed
+1 💚checkstyle0m 48smaster passed
+1 💚javadoc0m 51smaster passed
+0 🆗spotbugs3m 17sphoenix-core in master has 960 extant spotbugs warnings.
_ Patch Compile Tests _
+1 💚mvninstall9m 53sthe patch passed
+0hbaserecompile20m 12sHBase recompiled.
+1 💚compile1m 5sthe patch passed
+1 💚javac1m 5sthe patch passed
-1 ❌checkstyle0m 50sphoenix-core: The patch generated 18 new + 1122 unchanged - 11 fixed = 1140 total (was 1133)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 51sthe patch passed
+1 💚spotbugs3m 28sthe patch passed
_ Other Tests _
-1 ❌unit116m 22sphoenix-core in the patch failed.
+1 💚asflicense0m 34sThe patch does not generate ASF License warnings.
174m 15s
ReasonTests
Failed junit testsphoenix.end2end.PermissionsCacheIT
phoenix.end2end.index.IndexMetadataIT
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/3/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1197
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile
unameLinux 2e28931b7fed 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 / 84d317c
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-1197/3/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/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-1197/3/testReport/
Max. process+thread count11269 (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-1197/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 Apr 17, 2021

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 6sDocker 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.
_ master Compile Tests _
+0 🆗mvndep4m 50sMaven dependency ordering for branch
+1 💚mvninstall12m 41smaster passed
+0hbaserecompile24m 27sHBase recompiled.
+1 💚compile1m 33smaster passed
+1 💚checkstyle1m 0smaster passed
+1 💚javadoc1m 6smaster passed
+0 🆗spotbugs3m 18sphoenix-core in master has 960 extant spotbugs warnings.
_ Patch Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for patch
+1 💚mvninstall9m 58sthe patch passed
+0hbaserecompile16m 55sHBase recompiled.
+1 💚compile1m 34sthe patch passed
+1 💚javac1m 34sthe patch passed
-1 ❌checkstyle0m 49sphoenix-core: The patch generated 5 new + 1118 unchanged - 15 fixed = 1123 total (was 1133)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 1sThe patch has no ill-formed XML file.
+1 💚javadoc1m 7sthe patch passed
+1 💚spotbugs3m 28sthe patch passed
_ Other Tests _
-1 ❌unit115m 27sphoenix-core in the patch failed.
+1 💚unit2m 15sphoenix-server in the patch passed.
-1 ❌asflicense1m 1sThe patch generated 1 ASF License warnings.
178m 34s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/4/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1197
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile xml
unameLinux fb179d0badc1 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 / 84d317c
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-1197/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-1197/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-1197/4/testReport/
asflicensehttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/4/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count11696 (vs. ulimit of 30000)
modulesC: phoenix-core phoenix-server U: .
Console outputhttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/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 Apr 19, 2021

Copy link
Copy Markdown
Contributor

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 4sDocker 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.
_ master Compile Tests _
+1 💚mvninstall17m 29smaster passed
+0hbaserecompile25m 32sHBase recompiled.
+1 💚compile1m 4smaster passed
+1 💚checkstyle0m 50smaster passed
+1 💚javadoc0m 52smaster passed
+0 🆗spotbugs3m 16sphoenix-core in master has 960 extant spotbugs warnings.
_ Patch Compile Tests _
+1 💚mvninstall9m 57sthe patch passed
+0hbaserecompile20m 17sHBase recompiled.
+1 💚compile1m 5sthe patch passed
+1 💚javac1m 5sthe patch passed
-1 ❌checkstyle0m 49sphoenix-core: The patch generated 5 new + 1118 unchanged - 15 fixed = 1123 total (was 1133)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 51sthe patch passed
+1 💚spotbugs3m 25sthe patch passed
_ Other Tests _
-1 ❌unit116m 11sphoenix-core in the patch failed.
+1 💚asflicense0m 34sThe patch does not generate ASF License warnings.
175m 31s
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/5/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1197
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile
unameLinux 0da2e36e54a6 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 / f4ee13b
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-1197/5/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/5/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-1197/5/testReport/
Max. process+thread count12502 (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-1197/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.

@shahrs87

shahrs87 commented Apr 19, 2021

Copy link
Copy Markdown
ContributorAuthor

@gjacoby126 Could you please review the patch ?

All the unit tests are succeeding but unit test phase is failing with the following error.

[INFO] --- jacoco-maven-plugin:0.8.5:report (report) @ phoenix-core ---
[INFO] Loading execution data file /home/jenkins/jenkins-agent/workspace/enix-PreCommit-GitHub-PR_PR-1197/yetus-general-check/src/phoenix-core/target/jacoco.exec
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 01:55 h
[INFO] Finished at: 2021-04-19T16:52:02+00:00
[INFO] Final Memory: 78M/1632M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.jacoco:jacoco-maven-plugin:0.8.5:report (report) on project phoenix-core: An error has occurred in JaCoCo report generation. Error while creating report: malformed input around byte 2 -> [Help 1]
[ERROR]

@stoty

stoty commented Apr 19, 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 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 💚mvninstall16m 55smaster passed
+0hbaserecompile23m 56sHBase recompiled.
+1 💚compile1m 4smaster passed
+1 💚checkstyle0m 50smaster passed
+1 💚javadoc0m 52smaster passed
+0 🆗spotbugs3m 19sphoenix-core in master has 960 extant spotbugs warnings.
_ Patch Compile Tests _
+1 💚mvninstall9m 59sthe patch passed
+0hbaserecompile20m 25sHBase recompiled.
+1 💚compile1m 4sthe patch passed
+1 💚javac1m 4sthe patch passed
-1 ❌checkstyle0m 50sphoenix-core: The patch generated 2 new + 1118 unchanged - 15 fixed = 1120 total (was 1133)
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚javadoc0m 51sthe patch passed
+1 💚spotbugs3m 27sthe patch passed
_ Other Tests _
-1 ❌unit116m 22sphoenix-core in the patch failed.
+1 💚asflicense0m 34sThe patch does not generate ASF License warnings.
174m 25s
ReasonTests
Failed junit testsphoenix.end2end.index.IndexMetadataIT
SubsystemReport/Notes
DockerClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/6/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#1197
Optional Testsdupname asflicense javac javadoc unit spotbugs hbaserebuild hbaseanti checkstyle compile
unameLinux 0d935b4aa3a2 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 / f4ee13b
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-1197/6/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt
unithttps://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1197/6/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-1197/6/testReport/
Max. process+thread count11318 (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-1197/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.

@gjacoby126

gjacoby126 commented Apr 21, 2021

Copy link
Copy Markdown
Contributor

@shahrs87 - could you please check that IndexMetadataIT passes locally.

@shahrs87

shahrs87 commented Apr 21, 2021

Copy link
Copy Markdown
ContributorAuthor
[INFO] [INFO] --- maven-surefire-plugin:2.22.2:test (default-test) @ phoenix-core ---
[INFO] [INFO] -------------------------------------------------------
[INFO] T E S T S
[INFO] -------------------------------------------------------
[INFO] Running org.apache.phoenix.end2end.index.IndexMetadataIT
[INFO] Tests run: 21, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 214.302 s - in org.apache.phoenix.end2end.index.IndexMetadataIT
[INFO] [INFO] Results:
[INFO] [INFO] Tests run: 21, Failures: 0, Errors: 0, Skipped: 0

IndexMetadataIT succeeded locally. @gjacoby126

gjacoby126
gjacoby126 approved these changes Apr 21, 2021
View reviewed changes

@gjacoby126gjacoby126 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, just see comment on the other PR about harmonizing the commit names between branches, and with the JIRA

@gjacoby126
gjacoby126 merged commit 93db2fc into apache:masterApr 21, 2021
@shahrs87

shahrs87 commented Apr 21, 2021

Copy link
Copy Markdown
ContributorAuthor

Thank you @gjacoby126 for the review and merge !!

virajjasani pushed a commit that referenced this pull request May 28, 2021
@shahrs87Rushabh@virajjasani
PHOENIX-6444Extend Cell Tags to Delete object for Indexer coproc (#1197
71f4015
)
* PHOENIX-6444 Extend Cell Tags to Delete object for Indexer coproc
Co-authored-by: Rushabh <rushabh.shah@salesforce.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gjacoby126gjacoby126gjacoby126 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@shahrs87@stoty@gjacoby126