stoty commented on pull request #30: URL: https://github.com/apache/phoenix-connectors/pull/30#issuecomment-729646615
:broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |:----:|----------:|--------:|:--------| | +0 :ok: | reexec | 0m 27s | Docker mode activated. | ||| _ Prechecks _ | | +1 :green_heart: | dupname | 0m 0s | No case conflicting files found. | | +1 :green_heart: | hbaseanti | 0m 0s | Patch does not have any anti-patterns. | | +1 :green_heart: | @author | 0m 0s | The patch does not contain any @author tags. | | +1 :green_heart: | test4tests | 0m 0s | The patch appears to include 2 new or modified test files. | ||| _ master Compile Tests _ | | +0 :ok: | mvndep | 8m 38s | Maven dependency ordering for branch | | +1 :green_heart: | mvninstall | 7m 41s | master passed | | +1 :green_heart: | compile | 2m 28s | master passed | | +1 :green_heart: | checkstyle | 0m 19s | master passed | | -1 :x: | javadoc | 0m 14s | phoenix-kafka-base in master failed. | | -1 :x: | javadoc | 0m 9s | phoenix4-kafka in master failed. | | -1 :x: | javadoc | 0m 8s | phoenix5-kafka in master failed. | | +0 :ok: | spotbugs | 4m 24s | root in master has 111 extant spotbugs warnings. | | +0 :ok: | spotbugs | 1m 9s | phoenix-flume-base in master has 9 extant spotbugs warnings. | | +0 :ok: | spotbugs | 1m 40s | phoenix-hive-base in master has 47 extant spotbugs warnings. | | +0 :ok: | spotbugs | 0m 48s | phoenix-hive-base/phoenix4-hive in master has 47 extant spotbugs warnings. | | +0 :ok: | spotbugs | 0m 57s | phoenix-hive-base/phoenix5-hive in master has 46 extant spotbugs warnings. | | -1 :x: | spotbugs | 0m 23s | phoenix-kafka-base in master failed. | | +0 :ok: | spotbugs | 1m 5s | phoenix-pig-base in master has 12 extant spotbugs warnings. | | +0 :ok: | spotbugs | 1m 25s | phoenix-spark-base in master has 40 extant spotbugs warnings. | ||| _ Patch Compile Tests _ | | +0 :ok: | mvndep | 0m 34s | Maven dependency ordering for patch | | +1 :green_heart: | mvninstall | 6m 56s | the patch passed | | +1 :green_heart: | compile | 2m 8s | the patch passed | | +1 :green_heart: | javac | 2m 8s | root generated 0 new + 0 unchanged - 28 fixed = 0 total (was 28) | | +1 :green_heart: | checkstyle | 0m 16s | the patch passed | | -1 :x: | whitespace | 0m 0s | The patch has 8 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply | | +1 :green_heart: | xml | 0m 27s | The patch has no ill-formed XML file. | | +1 :green_heart: | yamllint | 0m 0s | The patch generated 0 new + 0 unchanged - 6 fixed = 0 total (was 6) | | +1 :green_heart: | javadoc | 0m 10s | phoenix-connectors-phoenix4-compat in the patch passed. | | +1 :green_heart: | javadoc | 0m 11s | phoenix4-flume in the patch passed. | | +1 :green_heart: | javadoc | 0m 11s | phoenix-pig-base_phoenix4-pig generated 0 new + 2 unchanged - 1 fixed = 2 total (was 3) | | -1 :x: | javadoc | 0m 9s | phoenix4-kafka in the patch failed. | | +1 :green_heart: | javadoc | 0m 13s | phoenix4-spark in the patch passed. | | +1 :green_heart: | javadoc | 0m 14s | phoenix-hive-base_phoenix4-hive generated 0 new + 3 unchanged - 1 fixed = 3 total (was 4) | | +1 :green_heart: | javadoc | 0m 8s | phoenix4-connectors-assembly in the patch passed. | | +1 :green_heart: | javadoc | 0m 10s | phoenix-connectors-phoenix5-compat in the patch passed. | | +1 :green_heart: | javadoc | 0m 12s | phoenix5-flume in the patch passed. | | +1 :green_heart: | javadoc | 0m 13s | phoenix-pig-base_phoenix5-pig generated 0 new + 2 unchanged - 1 fixed = 2 total (was 3) | | -1 :x: | javadoc | 0m 8s | phoenix5-kafka in the patch failed. | | +1 :green_heart: | javadoc | 0m 14s | phoenix5-spark in the patch passed. | | +1 :green_heart: | javadoc | 0m 15s | phoenix-hive-base_phoenix5-hive generated 0 new + 3 unchanged - 1 fixed = 3 total (was 4) | | -1 :x: | javadoc | 0m 11s | phoenix5-connectors-assembly in the patch failed. | | +1 :green_heart: | javadoc | 0m 48s | root generated 0 new + 20 unchanged - 4 fixed = 20 total (was 24) | | +1 :green_heart: | javadoc | 0m 15s | phoenix-flume-base in the patch passed. | | +1 :green_heart: | javadoc | 0m 24s | phoenix-hive-base generated 0 new + 6 unchanged - 2 fixed = 6 total (was 8) | | -1 :x: | javadoc | 0m 10s | phoenix-kafka-base in the patch failed. | | +1 :green_heart: | javadoc | 0m 15s | phoenix-pig-base generated 0 new + 4 unchanged - 2 fixed = 4 total (was 6) | | +1 :green_heart: | javadoc | 0m 18s | phoenix-spark-base in the patch passed. | | +1 :green_heart: | spotbugs | 0m 35s | phoenix-connectors-phoenix4-compat in the patch passed. | | +1 :green_heart: | spotbugs | 0m 57s | phoenix-hive-base/phoenix4-hive generated 0 new + 46 unchanged - 1 fixed = 46 total (was 47) | | +1 :green_heart: | spotbugs | 0m 36s | phoenix-connectors-phoenix5-compat in the patch passed. | | +1 :green_heart: | spotbugs | 1m 1s | phoenix-hive-base/phoenix5-hive generated 0 new + 45 unchanged - 1 fixed = 45 total (was 46) | | -1 :x: | spotbugs | 4m 11s | root generated 6 new + 110 unchanged - 1 fixed = 116 total (was 111) | | +1 :green_heart: | spotbugs | 1m 13s | phoenix-flume-base in the patch passed. | | +1 :green_heart: | spotbugs | 1m 31s | phoenix-hive-base generated 0 new + 46 unchanged - 1 fixed = 46 total (was 47) | | -1 :x: | spotbugs | 0m 25s | phoenix-kafka-base in the patch failed. | | -1 :x: | spotbugs | 1m 14s | phoenix-pig-base generated 6 new + 12 unchanged - 0 fixed = 18 total (was 12) | | +1 :green_heart: | spotbugs | 1m 29s | phoenix-spark-base in the patch passed. | ||| _ Other Tests _ | | -1 :x: | unit | 61m 55s | root in the patch failed. | | -1 :x: | asflicense | 2m 30s | The patch generated 12 ASF License warnings. | | | | 135m 3s | | | Reason | Tests | |-------:|:------| | FindBugs | module:root | | | Possible null pointer dereference in org.apache.phoenix.pig.PhoenixHBaseLoader.initializePhoenixPigConfiguration(String, Configuration) due to return value of called method Dereferenced at PhoenixHBaseLoader.java:org.apache.phoenix.pig.PhoenixHBaseLoader.initializePhoenixPigConfiguration(String, Configuration) due to return value of called method Dereferenced at PhoenixHBaseLoader.java:[line 147] | | | Possible null pointer dereference in org.apache.phoenix.pig.PhoenixHBaseStorage.setStoreLocation(String, Job) due to return value of called method Dereferenced at PhoenixHBaseStorage.java:org.apache.phoenix.pig.PhoenixHBaseStorage.setStoreLocation(String, Job) due to return value of called method Dereferenced at PhoenixHBaseStorage.java:[line 155] | | | Possible null pointer dereference in org.apache.phoenix.pig.util.PhoenixPigSchemaUtil.getResourceSchema(Configuration, PhoenixPigSchemaUtil$Dependencies) due to return value of called method Dereferenced at PhoenixPigSchemaUtil.java:org.apache.phoenix.pig.util.PhoenixPigSchemaUtil.getResourceSchema(Configuration, PhoenixPigSchemaUtil$Dependencies) due to return value of called method Dereferenced at PhoenixPigSchemaUtil.java:[line 68] | | | selectStatement must be non-null but is marked as nullable At QuerySchemaParserFunction.java:is marked as nullable At QuerySchemaParserFunction.java:[lines 56-88] | | | column must be non-null but is marked as nullable At QuerySchemaParserFunction.java:is marked as nullable At QuerySchemaParserFunction.java:[line 71] | | | columnProjector must be non-null but is marked as nullable At SqlQueryToColumnInfoFunction.java:is marked as nullable At SqlQueryToColumnInfoFunction.java:[line 62] | | FindBugs | module:phoenix-pig-base | | | Possible null pointer dereference in org.apache.phoenix.pig.PhoenixHBaseLoader.initializePhoenixPigConfiguration(String, Configuration) due to return value of called method Dereferenced at PhoenixHBaseLoader.java:org.apache.phoenix.pig.PhoenixHBaseLoader.initializePhoenixPigConfiguration(String, Configuration) due to return value of called method Dereferenced at PhoenixHBaseLoader.java:[line 147] | | | Possible null pointer dereference in org.apache.phoenix.pig.PhoenixHBaseStorage.setStoreLocation(String, Job) due to return value of called method Dereferenced at PhoenixHBaseStorage.java:org.apache.phoenix.pig.PhoenixHBaseStorage.setStoreLocation(String, Job) due to return value of called method Dereferenced at PhoenixHBaseStorage.java:[line 155] | | | Possible null pointer dereference in org.apache.phoenix.pig.util.PhoenixPigSchemaUtil.getResourceSchema(Configuration, PhoenixPigSchemaUtil$Dependencies) due to return value of called method Dereferenced at PhoenixPigSchemaUtil.java:org.apache.phoenix.pig.util.PhoenixPigSchemaUtil.getResourceSchema(Configuration, PhoenixPigSchemaUtil$Dependencies) due to return value of called method Dereferenced at PhoenixPigSchemaUtil.java:[line 68] | | | selectStatement must be non-null but is marked as nullable At QuerySchemaParserFunction.java:is marked as nullable At QuerySchemaParserFunction.java:[lines 56-88] | | | column must be non-null but is marked as nullable At QuerySchemaParserFunction.java:is marked as nullable At QuerySchemaParserFunction.java:[line 71] | | | columnProjector must be non-null but is marked as nullable At SqlQueryToColumnInfoFunction.java:is marked as nullable At SqlQueryToColumnInfoFunction.java:[line 62] | | Subsystem | Report/Notes | |----------:|:-------------| | Docker | ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/artifact/yetus-general-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/phoenix-connectors/pull/30 | | Optional Tests | dupname asflicense yamllint javac javadoc unit xml compile spotbugs hbaseanti checkstyle | | uname | Linux 41366cec35b2 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 tool | maven | | Personality | dev/phoenix-connectors-personality.sh | | git revision | master / 7a0f920 | | Default Java | Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08 | | javadoc | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/artifact/yetus-general-check/output/branch-javadoc-phoenix-kafka-base.txt | | javadoc | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/artifact/yetus-general-check/output/branch-javadoc-phoenix-kafka-base_phoenix4-kafka.txt | | javadoc | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/artifact/yetus-general-check/output/branch-javadoc-phoenix-kafka-base_phoenix5-kafka.txt | | spotbugs | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/artifact/yetus-general-check/output/branch-spotbugs-phoenix-kafka-base.txt | | whitespace | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/artifact/yetus-general-check/output/whitespace-eol.txt | | javadoc | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/artifact/yetus-general-check/output/patch-javadoc-phoenix-kafka-base_phoenix4-kafka.txt | | javadoc | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/artifact/yetus-general-check/output/patch-javadoc-phoenix-kafka-base_phoenix5-kafka.txt | | javadoc | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/artifact/yetus-general-check/output/patch-javadoc-phoenix5-connectors-assembly.txt | | javadoc | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/artifact/yetus-general-check/output/patch-javadoc-phoenix-kafka-base.txt | | spotbugs | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/artifact/yetus-general-check/output/new-spotbugs-root.html | | spotbugs | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/artifact/yetus-general-check/output/patch-spotbugs-phoenix-kafka-base.txt | | spotbugs | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/artifact/yetus-general-check/output/new-spotbugs-phoenix-pig-base.html | | unit | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/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-30/12/testReport/ | | asflicense | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/artifact/yetus-general-check/output/patch-asflicense-problems.txt | | Max. process+thread count | 3292 (vs. ulimit of 30000) | | modules | C: phoenix-connectors-phoenix4-compat phoenix-flume-base/phoenix4-flume phoenix-pig-base/phoenix4-pig phoenix-kafka-base/phoenix4-kafka phoenix-spark-base/phoenix4-spark phoenix-hive-base/phoenix4-hive phoenix4-connectors-assembly phoenix-connectors-phoenix5-compat phoenix-flume-base/phoenix5-flume phoenix-pig-base/phoenix5-pig phoenix-kafka-base/phoenix5-kafka phoenix-spark-base/phoenix5-spark phoenix-hive-base/phoenix5-hive phoenix5-connectors-assembly . phoenix-flume-base phoenix-hive-base phoenix-kafka-base phoenix-pig-base phoenix-spark-base U: . | | Console output | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-30/12/console | | versions | git=2.7.4 maven=3.3.9 spotbugs=4.1.4 yamllint=1.20.0 | | Powered by | Apache Yetus 0.12.0 https://yetus.apache.org | This message was automatically generated. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
