stoty commented on pull request #995: URL: https://github.com/apache/phoenix/pull/995#issuecomment-737236983
:broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Comment | |:----:|----------:|--------:|:--------| | +0 :ok: | reexec | 0m 35s | 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. | ||| _ 4.x-PHOENIX-5182 Compile Tests _ | | +1 :green_heart: | mvninstall | 10m 50s | 4.x-PHOENIX-5182 passed | | +1 :green_heart: | compile | 0m 56s | 4.x-PHOENIX-5182 passed | | +1 :green_heart: | checkstyle | 1m 15s | 4.x-PHOENIX-5182 passed | | +1 :green_heart: | javadoc | 0m 44s | 4.x-PHOENIX-5182 passed | | +0 :ok: | spotbugs | 2m 59s | phoenix-core in 4.x-PHOENIX-5182 has 954 extant spotbugs warnings. | ||| _ Patch Compile Tests _ | | +1 :green_heart: | mvninstall | 5m 28s | the patch passed | | +1 :green_heart: | compile | 0m 54s | the patch passed | | +1 :green_heart: | javac | 0m 54s | the patch passed | | -1 :x: | checkstyle | 1m 15s | phoenix-core: The patch generated 168 new + 1770 unchanged - 25 fixed = 1938 total (was 1795) | | +1 :green_heart: | whitespace | 0m 0s | The patch has no whitespace issues. | | +1 :green_heart: | javadoc | 0m 44s | the patch passed | | -1 :x: | spotbugs | 3m 20s | phoenix-core generated 5 new + 954 unchanged - 0 fixed = 959 total (was 954) | ||| _ Other Tests _ | | -1 :x: | unit | 131m 40s | phoenix-core in the patch failed. | | -1 :x: | asflicense | 0m 38s | The patch generated 1 ASF License warnings. | | | | 164m 15s | | | Reason | Tests | |-------:|:------| | FindBugs | module:phoenix-core | | | Repeated conditional test in org.apache.phoenix.coprocessor.IndexToolVerificationResult$PhaseResult.equals(Object) At IndexToolVerificationResult.java: At IndexToolVerificationResult.java:[line 222] | | | org.apache.phoenix.mapreduce.index.IndexVerificationResultRepository.AFTER_REPAIR_EXTRA_UNVERIFIED_INDEX_ROW_COUNT_BYTES is a mutable array At IndexVerificationResultRepository.java: At IndexVerificationResultRepository.java:[line 136] | | | org.apache.phoenix.mapreduce.index.IndexVerificationResultRepository.AFTER_REPAIR_EXTRA_VERIFIED_INDEX_ROW_COUNT_BYTES is a mutable array At IndexVerificationResultRepository.java: At IndexVerificationResultRepository.java:[line 133] | | | org.apache.phoenix.mapreduce.index.IndexVerificationResultRepository.BEFORE_REPAIR_EXTRA_UNVERIFIED_INDEX_ROW_COUNT_BYTES is a mutable array At IndexVerificationResultRepository.java: At IndexVerificationResultRepository.java:[line 129] | | | org.apache.phoenix.mapreduce.index.IndexVerificationResultRepository.BEFORE_REPAIR_EXTRA_VERIFIED_INDEX_ROW_COUNT_BYTES is a mutable array At IndexVerificationResultRepository.java: At IndexVerificationResultRepository.java:[line 126] | | Failed junit tests | phoenix.end2end.InQueryIT | | | phoenix.end2end.TenantSpecificViewIndexIT | | | phoenix.end2end.IndexRepairRegionScannerIT | | | phoenix.end2end.RangeScanIT | | | phoenix.end2end.StoreNullsIT | | 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-995/2/artifact/yetus-general-check/output/Dockerfile | | GITHUB PR | https://github.com/apache/phoenix/pull/995 | | Optional Tests | dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile | | uname | Linux 7b4268fae47f 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 | 4.x-PHOENIX-5182 / 92c8579 | | Default Java | Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08 | | checkstyle | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-995/2/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-995/2/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-995/2/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-995/2/testReport/ | | asflicense | https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-995/2/artifact/yetus-general-check/output/patch-asflicense-problems.txt | | Max. process+thread count | 7091 (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-995/2/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]
