PHOENIX-3136 Do not relocate org.apache.calcite in queryserver/thin-client Relocation of these classes breaks compatibility with previous versions of Phoenix.
Project: http://git-wip-us.apache.org/repos/asf/phoenix/repo Commit: http://git-wip-us.apache.org/repos/asf/phoenix/commit/c357318b Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/c357318b Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/c357318b Branch: refs/heads/4.8-HBase-1.2 Commit: c357318bd6e4cceee5d55a8e2dd76f9b313301c6 Parents: 5b5f0c9 Author: Josh Elser <els...@apache.org> Authored: Tue Aug 2 11:51:37 2016 -0400 Committer: Josh Elser <els...@apache.org> Committed: Tue Aug 2 19:17:14 2016 -0400 ---------------------------------------------------------------------- phoenix-queryserver-client/pom.xml | 7 +++---- phoenix-queryserver/pom.xml | 7 +++---- 2 files changed, 6 insertions(+), 8 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/phoenix/blob/c357318b/phoenix-queryserver-client/pom.xml ---------------------------------------------------------------------- diff --git a/phoenix-queryserver-client/pom.xml b/phoenix-queryserver-client/pom.xml index 8b2ecc8..ecc9edd 100644 --- a/phoenix-queryserver-client/pom.xml +++ b/phoenix-queryserver-client/pom.xml @@ -113,10 +113,9 @@ <shadedPattern>${shaded.package}.com.fasterxml</shadedPattern> </relocation> <!-- ORG relocation --> - <relocation> - <pattern>org.apache.calcite</pattern> - <shadedPattern>${shaded.package}.org.apache.calcite</shadedPattern> - </relocation> + <!-- Calcite/Avatica is not relocated because the wire API (as of <=1.8.0) expects + consistent class names on client and server. Relocating these would break + backwards compatibility. --> <relocation> <pattern>org.apache.commons</pattern> <shadedPattern>${shaded.package}.org.apache.commons</shadedPattern> http://git-wip-us.apache.org/repos/asf/phoenix/blob/c357318b/phoenix-queryserver/pom.xml ---------------------------------------------------------------------- diff --git a/phoenix-queryserver/pom.xml b/phoenix-queryserver/pom.xml index f09e1cb..1c186a1 100644 --- a/phoenix-queryserver/pom.xml +++ b/phoenix-queryserver/pom.xml @@ -105,10 +105,9 @@ <pattern>org.eclipse.jetty</pattern> <shadedPattern>${shaded.package}.org.eclipse.jetty</shadedPattern> </relocation> - <relocation> - <pattern>org.apache.calcite</pattern> - <shadedPattern>${shaded.package}.org.apache.calcite</shadedPattern> - </relocation> + <!-- Calcite/Avatica is not relocated because the wire API (as of <=1.8.0) expects + consistent class names on client and server. Relocating these would break + backwards compatibility. --> </relocations> </configuration> </execution>
