stoty commented on pull request #1233:
URL: https://github.com/apache/phoenix/pull/1233#issuecomment-842733351


   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |   5m 38s |  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 :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.  
|
   ||| _ 4.x Compile Tests _ |
   | +0 :ok: |  mvndep  |   5m  9s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  10m 27s |  4.x passed  |
   | +1 :green_heart: |  compile  |   1m 39s |  4.x passed  |
   | +1 :green_heart: |  checkstyle  |   0m 44s |  4.x passed  |
   | +1 :green_heart: |  javadoc  |   1m  2s |  4.x passed  |
   | +0 :ok: |  spotbugs  |   3m 18s |  phoenix-core in 4.x has 951 extant 
spotbugs warnings.  |
   | +0 :ok: |  spotbugs  |   0m 46s |  phoenix-tools in 4.x has 5 extant 
spotbugs warnings.  |
   ||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 11s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   6m 50s |  the patch passed  |
   | +1 :green_heart: |  compile  |   1m 40s |  the patch passed  |
   | +1 :green_heart: |  javac  |   1m 40s |  the patch passed  |
   | -1 :x: |  checkstyle  |   0m 32s |  phoenix-core: The patch generated 17 
new + 294 unchanged - 13 fixed = 311 total (was 307)  |
   | -1 :x: |  checkstyle  |   0m 11s |  phoenix-tools: The patch generated 17 
new + 112 unchanged - 8 fixed = 129 total (was 120)  |
   | -1 :x: |  whitespace  |   0m  0s |  The patch 2 line(s) with tabs.  |
   | +1 :green_heart: |  javadoc  |   1m  2s |  the patch passed  |
   | +1 :green_heart: |  spotbugs  |   4m 27s |  the patch passed  |
   ||| _ Other Tests _ |
   | -1 :x: |  unit  | 196m 27s |  phoenix-core in the patch failed.  |
   | -1 :x: |  unit  |   4m 40s |  phoenix-tools in the patch failed.  |
   | -1 :x: |  asflicense  |   1m 11s |  The patch generated 1 ASF License 
warnings.  |
   |  |   | 248m 56s |   |
   
   
   | Reason | Tests |
   |-------:|:------|
   | Failed junit tests | phoenix.end2end.AuditLoggingIT |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1233/2/artifact/yetus-general-check/output/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/phoenix/pull/1233 |
   | JIRA Issue | PHOENIX-6454 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs hbaseanti 
checkstyle compile |
   | uname | Linux cf672e60ed2f 4.15.0-142-generic #146-Ubuntu SMP Tue Apr 13 
01:11:19 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev/phoenix-personality.sh |
   | git revision | 4.x / 69a9ec3 |
   | 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-1233/2/artifact/yetus-general-check/output/diff-checkstyle-phoenix-core.txt
 |
   | checkstyle | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1233/2/artifact/yetus-general-check/output/diff-checkstyle-phoenix-tools.txt
 |
   | whitespace | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1233/2/artifact/yetus-general-check/output/whitespace-tabs.txt
 |
   | unit | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1233/2/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt
 |
   | unit | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1233/2/artifact/yetus-general-check/output/patch-unit-phoenix-tools.txt
 |
   |  Test Results | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1233/2/testReport/
 |
   | asflicense | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1233/2/artifact/yetus-general-check/output/patch-asflicense-problems.txt
 |
   | Max. process+thread count | 5163 (vs. ulimit of 30000) |
   | modules | C: phoenix-core phoenix-tools U: . |
   | Console output | 
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1233/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]


Reply via email to