You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@phoenix.apache.org by el...@apache.org on 2016/08/02 23:25:33 UTC
[7/8] phoenix git commit: PHOENIX-3136 Do not relocate
org.apache.calcite in queryserver/thin-client
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/ce572433
Tree: http://git-wip-us.apache.org/repos/asf/phoenix/tree/ce572433
Diff: http://git-wip-us.apache.org/repos/asf/phoenix/diff/ce572433
Branch: refs/heads/4.8-HBase-1.0
Commit: ce572433dcc80d14d924c4d0b04d23692f391c7f
Parents: 81ffc48
Author: Josh Elser <el...@apache.org>
Authored: Tue Aug 2 11:51:37 2016 -0400
Committer: Josh Elser <el...@apache.org>
Committed: Tue Aug 2 19:21:17 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/ce572433/phoenix-queryserver-client/pom.xml
----------------------------------------------------------------------
diff --git a/phoenix-queryserver-client/pom.xml b/phoenix-queryserver-client/pom.xml
index d64cc3e..c2e8920 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/ce572433/phoenix-queryserver/pom.xml
----------------------------------------------------------------------
diff --git a/phoenix-queryserver/pom.xml b/phoenix-queryserver/pom.xml
index 599ed7d..441c6d5 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>