[
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)