phoenix-connectors icon indicating copy to clipboard operation
phoenix-connectors copied to clipboard

PHOENIX-6166 - Make Tephra support optional for phoenix 5 connectors

Open richardantal opened this issue 3 years ago • 2 comments

richardantal avatar Aug 27 '21 11:08 richardantal

:broken_heart: -1 overall

Vote Subsystem Runtime Comment
+0 :ok: reexec 1m 1s Docker mode activated.
_ Prechecks _
+1 :green_heart: dupname 0m 0s No case conflicting files found.
+1 :green_heart: @author 0m 0s The patch does not contain any @author tags.
-1 :x: test4tests 0m 0s The 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 :ok: mvndep 10m 36s Maven dependency ordering for branch
+1 :green_heart: mvninstall 16m 41s master passed
+1 :green_heart: compile 2m 37s master passed
+1 :green_heart: javadoc 1m 59s master passed
_ Patch Compile Tests _
+0 :ok: mvndep 0m 20s Maven dependency ordering for patch
+1 :green_heart: mvninstall 16m 32s the patch passed
+1 :green_heart: compile 2m 34s the patch passed
+1 :green_heart: javac 2m 34s the patch passed
+1 :green_heart: whitespace 0m 0s The patch has no whitespace issues.
+1 :green_heart: xml 0m 8s The patch has no ill-formed XML file.
+1 :green_heart: javadoc 2m 1s the patch passed
_ Other Tests _
-1 :x: unit 100m 2s root in the patch failed.
-1 :x: asflicense 0m 53s The patch generated 7 ASF License warnings.
156m 17s
Reason Tests
Failed junit tests phoenix.spark.OrderByIT
phoenix.spark.SaltedTableIT
phoenix.spark.AggregateIT
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-62/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR https://github.com/apache/phoenix-connectors/pull/62
Optional Tests dupname asflicense javac javadoc unit xml compile
uname Linux db07fe0bc759 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 tool maven
Personality dev/phoenix-connectors-personality.sh
git revision master / 349a2b2
Default Java Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unit https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-62/1/artifact/yetus-general-check/output/patch-unit-root.txt
Test Results https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-62/1/testReport/
asflicense https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-62/1/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count 1897 (vs. ulimit of 30000)
modules C: phoenix-hive-base/phoenix5-hive phoenix-flume-base/phoenix5-flume phoenix-pig-base/phoenix5-pig phoenix-kafka-base/phoenix5-kafka phoenix-spark-base/phoenix5-spark . U: .
Console output https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-62/1/console
versions git=2.7.4 maven=3.3.9
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

stoty avatar Aug 27 '21 14:08 stoty

:broken_heart: -1 overall

Vote Subsystem Runtime Comment
+0 :ok: reexec 6m 45s Docker mode activated.
_ Prechecks _
+1 :green_heart: dupname 0m 0s No case conflicting files found.
+1 :green_heart: @author 0m 0s The patch does not contain any @author tags.
-1 :x: test4tests 0m 0s The 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 :ok: mvndep 10m 43s Maven dependency ordering for branch
+1 :green_heart: mvninstall 20m 15s master passed
+1 :green_heart: compile 2m 52s master passed
+1 :green_heart: javadoc 1m 57s master passed
_ Patch Compile Tests _
+0 :ok: mvndep 0m 20s Maven dependency ordering for patch
+1 :green_heart: mvninstall 19m 58s the patch passed
+1 :green_heart: compile 2m 46s the patch passed
+1 :green_heart: javac 2m 46s the patch passed
+1 :green_heart: whitespace 0m 0s The patch has no whitespace issues.
+1 :green_heart: xml 0m 7s The patch has no ill-formed XML file.
+1 :green_heart: javadoc 1m 59s the patch passed
_ Other Tests _
-1 :x: unit 79m 43s root in the patch failed.
-1 :x: asflicense 0m 50s The patch generated 7 ASF License warnings.
149m 5s
Reason Tests
Failed junit tests phoenix.spark.OrderByIT
phoenix.spark.AggregateIT
phoenix.spark.SaltedTableIT
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-62/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR https://github.com/apache/phoenix-connectors/pull/62
Optional Tests dupname asflicense javac javadoc unit xml compile
uname Linux c11ba92adbd0 4.15.0-147-generic #151-Ubuntu SMP Fri Jun 18 19:21:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev/phoenix-connectors-personality.sh
git revision master / 349a2b2
Default Java Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08
unit https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-62/2/artifact/yetus-general-check/output/patch-unit-root.txt
Test Results https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-62/2/testReport/
asflicense https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-62/2/artifact/yetus-general-check/output/patch-asflicense-problems.txt
Max. process+thread count 1805 (vs. ulimit of 30000)
modules C: phoenix-hive-base/phoenix5-hive phoenix-flume-base/phoenix5-flume phoenix-pig-base/phoenix5-pig phoenix-kafka-base/phoenix5-kafka phoenix-spark-base/phoenix5-spark . U: .
Console output https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-62/2/console
versions git=2.7.4 maven=3.3.9
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

stoty avatar Aug 27 '21 17:08 stoty

Another version has been committed in the meantime.

stoty avatar Jul 25 '24 06:07 stoty