[ 
https://issues.apache.org/jira/browse/PHOENIX-6010?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17198335#comment-17198335
 ] 

Hadoop QA commented on PHOENIX-6010:
------------------------------------

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  7m 
22s{color} | {color:blue} Docker mode activated. {color} |
|| || || || {color:brown} Prechecks {color} ||
| {color:green}+1{color} | {color:green} dupname {color} | {color:green}  0m  
5s{color} | {color:green} No case conflicting files found. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green}  0m 
 0s{color} | {color:green} The patch appears to include 87 new or modified test 
files. {color} |
|| || || || {color:brown} master Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
35s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  9m 
56s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m 
28s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green} 23m 
 8s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  3m  
5s{color} | {color:green} master passed {color} |
| {color:blue}0{color} | {color:blue} spotbugs {color} | {color:blue} 32m 
23s{color} | {color:blue} Both FindBugs and SpotBugs are enabled, using 
SpotBugs. {color} |
| {color:red}-1{color} | {color:red} spotbugs {color} | {color:red}  4m 
34s{color} | {color:red} phoenix-core in master failed. {color} |
| {color:blue}0{color} | {color:blue} spotbugs {color} | {color:blue}  0m 
53s{color} | {color:blue} phoenix-pherf in master has 42 extant spotbugs 
warnings. {color} |
| {color:blue}0{color} | {color:blue} spotbugs {color} | {color:blue}  0m 
42s{color} | {color:blue} phoenix-tools in master has 3 extant spotbugs 
warnings. {color} |
| {color:red}-1{color} | {color:red} spotbugs {color} | {color:red}  0m  
7s{color} | {color:red} root in master failed. {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
53s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green} 11m 
54s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m 
39s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  1m 
39s{color} | {color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red} 24m 
58s{color} | {color:red} root: The patch generated 521 new + 54753 unchanged - 
516 fixed = 55274 total (was 55269) {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} xml {color} | {color:green}  0m  
8s{color} | {color:green} The patch has no ill-formed XML file. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  3m 
42s{color} | {color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} spotbugs {color} | {color:red}  5m 
11s{color} | {color:red} phoenix-core in the patch failed. {color} |
| {color:red}-1{color} | {color:red} spotbugs {color} | {color:red}  0m  
9s{color} | {color:red} root in the patch failed. {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:red}-1{color} | {color:red} unit {color} | {color:red}172m 49s{color} 
| {color:red} root in the patch failed. {color} |
| {color:red}-1{color} | {color:red} asflicense {color} | {color:red}  1m 
18s{color} | {color:red} The patch generated 1 ASF License warnings. {color} |
| {color:black}{color} | {color:black} {color} | {color:black}281m 12s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | ClientAPI=1.40 ServerAPI=1.40 base: 
https://ci-hadoop.apache.org/job/Phoenix/job/Jenkins-yetus-precommit/21/artifact/patchprocess/Dockerfile
 |
| JIRA Issue | PHOENIX-6010 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/13011674/PHOENIX-6010.master.v9.patch
 |
| Optional Tests | dupname asflicense javac javadoc unit xml compile spotbugs 
findbugs hbaseanti checkstyle |
| uname | Linux 42ec667e8b06 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 / cd42483 |
| Default Java | Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08 |
| spotbugs | 
https://ci-hadoop.apache.org/job/Phoenix/job/Jenkins-yetus-precommit/21/artifact/patchprocess/branch-spotbugs-phoenix-core.txt
 |
| spotbugs | 
https://ci-hadoop.apache.org/job/Phoenix/job/Jenkins-yetus-precommit/21/artifact/patchprocess/branch-spotbugs-root.txt
 |
| checkstyle | 
https://ci-hadoop.apache.org/job/Phoenix/job/Jenkins-yetus-precommit/21/artifact/patchprocess/diff-checkstyle-root.txt
 |
| spotbugs | 
https://ci-hadoop.apache.org/job/Phoenix/job/Jenkins-yetus-precommit/21/artifact/patchprocess/patch-spotbugs-phoenix-core.txt
 |
| spotbugs | 
https://ci-hadoop.apache.org/job/Phoenix/job/Jenkins-yetus-precommit/21/artifact/patchprocess/patch-spotbugs-root.txt
 |
| unit | 
https://ci-hadoop.apache.org/job/Phoenix/job/Jenkins-yetus-precommit/21/artifact/patchprocess/patch-unit-root.txt
 |
|  Test Results | 
https://ci-hadoop.apache.org/job/Phoenix/job/Jenkins-yetus-precommit/21/testReport/
 |
| asflicense | 
https://ci-hadoop.apache.org/job/Phoenix/job/Jenkins-yetus-precommit/21/artifact/patchprocess/patch-asflicense-problems.txt
 |
| Max. process+thread count | 6274 (vs. ulimit of 12500) |
| modules | C: phoenix-core phoenix-pherf phoenix-client phoenix-tools . U: . |
| Console output | 
https://ci-hadoop.apache.org/job/Phoenix/job/Jenkins-yetus-precommit/21/console 
|
| versions | git=2.7.4 maven=3.3.9 spotbugs=4.1.2 |
| Powered by | Apache Yetus 0.12.0 https://yetus.apache.org |


This message was automatically generated.



> Create phoenix-thirdparty, and consume guava through it
> -------------------------------------------------------
>
>                 Key: PHOENIX-6010
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-6010
>             Project: Phoenix
>          Issue Type: Improvement
>          Components: core, omid, tephra
>    Affects Versions: 5.1.0, 4.16.0
>            Reporter: Istvan Toth
>            Assignee: Istvan Toth
>            Priority: Major
>         Attachments: PHOENIX-6010.master.v1.patch, 
> PHOENIX-6010.master.v2.patch, PHOENIX-6010.master.v3.patch, 
> PHOENIX-6010.master.v4.patch, PHOENIX-6010.master.v5.patch, 
> PHOENIX-6010.master.v6.patch, PHOENIX-6010.master.v7.patch, 
> PHOENIX-6010.master.v8.patch, PHOENIX-6010.master.v9.patch
>
>          Time Spent: 1h 50m
>  Remaining Estimate: 0h
>
> We have long-standing and well-documented problems with Guava, just like the 
> rest of the Hadoop components.
> Adopt the solution used by HBase:
>  * create phoenix-thirdparty repo
>  * create a pre-shaded phoenix-shaded-guava artifact in it
>  * Use the pre-shaded Guava in every phoenix component
> The advantages are well-known, but to name a few:
>  * Phoenix will work with Hadoop 3.1.3+
>  * One less CVE in our direct dependencies
>  * No more conflict with our consumer's Guava versions



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to