stoty commented on pull request #970: URL: https://github.com/apache/phoenix/pull/970#issuecomment-728298122
:broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |:----:|----------:|--------:|:--------| | +0 :ok: | reexec | 1m 14s | 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 1 new or modified test files. | ||| _ master Compile Tests _ | | -1 :x: | mvninstall | 3m 58s | root in master failed. | | +1 :green_heart: | compile | 1m 4s | master passed | | +1 :green_heart: | checkstyle | 3m 14s | master passed | | +1 :green_heart: | javadoc | 0m 47s | master passed | | +0 :ok: | spotbugs | 3m 8s | phoenix-core in master has 962 extant spotbugs warnings. | ||| _ Patch Compile Tests _ | | -1 :x: | mvninstall | 1m 8s | root in the patch failed. | | +1 :green_heart: | compile | 0m 57s | the patch passed | | +1 :green_heart: | javac | 0m 57s | the patch passed | | -1 :x: | checkstyle | 3m 18s | phoenix-core: The patch generated 254 new + 9998 unchanged - 118 fixed = 10252 total (was 10116) | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | +1 :green_heart: | javadoc | 0m 45s | the patch passed | | -1 :x: | spotbugs | 3m 24s | phoenix-core generated 6 new + 958 unchanged - 4 fixed = 964 total (was 962) | ||| _ Other Tests _ | | -1 :x: | unit | 168m 13s | phoenix-core in the patch failed. | | +1 :green_heart: | asflicense | 0m 24s | The patch does not generate ASF License warnings. | | | | 194m 18s | | | Reason | Tests | |-------:|:------| | FindBugs | module:phoenix-core | | | org.apache.phoenix.jdbc.PhoenixDatabaseMetaData.SYSTEM_TASK_HISTORY_NAME_BYTES is a mutable array At StatisticsUtil.java: At StatisticsUtil.java:[line 78] | | | org.apache.phoenix.jdbc.PhoenixDatabaseMetaData.SYSTEM_TASK_QUEUE_NAME_BYTES is a mutable array At StatisticsUtil.java: At StatisticsUtil.java:[line 77] | | | org.apache.phoenix.query.ConnectionQueryServicesImpl.createOtherSystemTables(PhoenixConnection) may fail to clean up java.sql.Statement Obligation to clean up resource created at ConnectionQueryServicesImpl.java:up java.sql.Statement Obligation to clean up resource created at ConnectionQueryServicesImpl.java:[line 3449] is not discharged | | | Exception is caught when Exception is not thrown in org.apache.phoenix.query.ConnectionQueryServicesImpl.upgradeSystemTask(PhoenixConnection, Map) At ConnectionQueryServicesImpl.java:is not thrown in org.apache.phoenix.query.ConnectionQueryServicesImpl.upgradeSystemTask(PhoenixConnection, Map) At ConnectionQueryServicesImpl.java:[line 4217] | | | org.apache.phoenix.query.ConnectionlessQueryServicesImpl.init(String, Properties) may fail to clean up java.sql.Statement Obligation to clean up resource created at ConnectionlessQueryServicesImpl.java:clean up java.sql.Statement Obligation to clean up resource created at ConnectionlessQueryServicesImpl.java:[line 382] is not discharged | | | org.apache.phoenix.schema.task.Task.deleteTask(PhoenixConnection, PTable$TaskType, Timestamp, String, String, String, boolean, String) may fail to clean up java.sql.Statement on checked exception Obligation to clean up resource created at Task.java:String, boolean, String) may fail to clean up java.sql.Statement on checked exception Obligation to clean up resource created at Task.java:[line 191] is not discharged | | Failed junit tests | phoenix.end2end.TableSnapshotReadsMapReduceIT | | | phoenix.end2end.SystemTablesCreationOnConnectionIT | | | phoenix.end2end.MigrateSystemTablesToSystemNamespaceIT | | | phoenix.end2end.UpsertSelectIT | | Subsystem | Report/Notes | |----------:|:-------------| | Docker | ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/artifact/yetus-general-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/phoenix/pull/970 | | Optional Tests | dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile | | uname | Linux 79b808f9bbf9 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-personality.sh | | git revision | master / 5ff3edb | | Default Java | Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08 | | mvninstall | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/artifact/yetus-general-check/output/branch-mvninstall-root.txt | | mvninstall | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/artifact/yetus-general-check/output/patch-mvninstall-root.txt | | checkstyle | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt | | spotbugs | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/artifact/yetus-general-check/output/new-spotbugs-phoenix-core.html | | unit | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt | | Test Results | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/testReport/ | | Max. process+thread count | 6361 (vs. ulimit of 30000) | | modules | C: phoenix-core U: phoenix-core | | Console output | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-970/1/console | | versions | git=2.7.4 maven=3.3.9 spotbugs=4.1.3 | | 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]
