You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@phoenix.apache.org by Istvan Toth <st...@apache.org> on 2020/09/24 03:54:14 UTC

Please don't use unshaded Guava classes in Phoenix master

Hi!

We have recently switched to shaded Guava from phoenix-thirdparty in master.

Using unshaded Guava will break maven verify (and cause dependency problems
down the line), so please take care not to commit patches that use unshaded
Guava classes.

Yetus does flag these patches with -1.

I know that the tests are in bad shape, and the IT results are kind of
useless in the PreCommit check right now, but Yetus flags this problem as
an *mvninstall* error, which should be taken seriously:

   Patch Compile Tests
-1 mvninstall 1m 14s root in the patch failed.

you'll see the following line in the linked mvn install log:

[WARNING] Used undeclared dependencies found:
[WARNING]    com.google.guava:guava:jar:11.0.2:compile


regards
Istvan