calcite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From James Taylor <jamestay...@apache.org>
Subject Re: [VOTE] apache-calcite-avatica-1.7.0 (release candidate 1)
Date Fri, 11 Mar 2016 20:21:26 GMT
+1

Successfully ran phoenix-server and phoenix-server-client unit tests with
poms modified to point to new avatica only artifact (see below - will file
phoenix jira with these changes too).

Thanks,
James

diff --git a/phoenix-server-client/pom.xml b/phoenix-server-client/pom.xml
index e9f4bdb..95184c2 100644
--- a/phoenix-server-client/pom.xml
+++ b/phoenix-server-client/pom.xml
@@ -54,8 +54,9 @@

   <dependencies>
     <dependency>
-      <groupId>org.apache.calcite</groupId>
-      <artifactId>calcite-avatica</artifactId>
+      <groupId>org.apache.calcite.avatica</groupId>
+      <artifactId>avatica</artifactId>
+      <version>${avatica.version}</version>
     </dependency>
     <dependency>
       <!-- a dependency for the thin-client uberjar -->
diff --git a/phoenix-server/pom.xml b/phoenix-server/pom.xml
index 9eb0e33..536bf02 100644
--- a/phoenix-server/pom.xml
+++ b/phoenix-server/pom.xml
@@ -66,14 +66,14 @@
       <artifactId>phoenix-server-client</artifactId>
     </dependency>
     <dependency>
-      <groupId>org.apache.calcite</groupId>
-      <artifactId>calcite-avatica</artifactId>
-      <version>${calcite.version}</version>
+      <groupId>org.apache.calcite.avatica</groupId>
+      <artifactId>avatica</artifactId>
+      <version>${avatica.version}</version>
     </dependency>
     <dependency>
-      <groupId>org.apache.calcite</groupId>
-      <artifactId>calcite-avatica-server</artifactId>
-      <version>${calcite.version}</version>
+      <groupId>org.apache.calcite.avatica</groupId>
+      <artifactId>avatica-server</artifactId>
+      <version>${avatica.version}</version>
     </dependency>
     <dependency>
       <groupId>org.apache.phoenix</groupId>
diff --git a/pom.xml b/pom.xml
index 1c7dc53..69b52ea 100644
--- a/pom.xml
+++ b/pom.xml
@@ -51,6 +51,10 @@
       </snapshots>
     </repository>
     <repository>
+      <id>apache calcite avatica RC</id>
+      <url>
https://repository.apache.org/content/repositories/orgapachecalcite-1014
</url>
+    </repository>
+    <repository>
       <id>sonatype-nexus-snapshots</id>
       <name>Sonatype Nexus Snapshots</name>
       <url>https://oss.sonatype.org/content/repositories/snapshots</url>
@@ -110,7 +114,7 @@
     <!-- Do not change jodatime.version until HBASE-15199 is fixed -->
     <jodatime.version>1.6</jodatime.version>
     <joni.version>2.1.2</joni.version>
-    <calcite.version>1.6.0</calcite.version>
+    <avatica.version>1.7.0</avatica.version>
     <jettyVersion>8.1.7.v20120910</jettyVersion>
     <tephra.version>0.7.0</tephra.version>
     <spark.version>1.5.2</spark.version>
@@ -590,14 +594,9 @@
         </exclusions>
       </dependency>
       <dependency>
-        <groupId>org.apache.calcite</groupId>
-        <artifactId>calcite-avatica</artifactId>
-        <version>${calcite.version}</version>
-      </dependency>
-      <dependency>
-        <groupId>org.apache.calcite</groupId>
-        <artifactId>calcite-avatica-server</artifactId>
-        <version>${calcite.version}</version>
+        <groupId>org.apache.calcite.avatica</groupId>
+        <artifactId>avatica</artifactId>
+        <version>${avatica.version}</version>
       </dependency>

       <!-- Transaction dependencies -->


On Fri, Mar 11, 2016 at 11:32 AM, Josh Elser <josh.elser@gmail.com> wrote:

> Actually, "calcite-avatica" => "avatica".
>
> Just drop "calcite-" from the artifactIds, and add a ".avatica" to the
> groupId.
>
> After we get this released, I was planning to send a PR over to Apache
> Phoenix-landia, James. Thanks for trying to test this out proactively
> though.
>
>
> Julian Hyde wrote:
>
>> “calcite-avatica-server” =>  “avatica-server”
>>
>> “calcite-avatica” =>  “avatica-core”
>>
>> There is now also an “avatica-parent” but I don’t think you should use it.
>>
>> Julian
>>
>>
>> On Mar 11, 2016, at 10:45 AM, James Taylor<jamestaylor@apache.org>
>>> wrote:
>>>
>>> I made that change, but I'm a bit confused about what to use for the
>>> artifactId. Here's what I did:
>>> -<groupId>org.apache.calcite</groupId>
>>> -<artifactId>calcite-avatica-server</artifactId>
>>> +<groupId>org.apache.calcite.avatica</groupId>
>>> +<artifactId>avatica-parent</artifactId>
>>>
>>> Complete change here:
>>>
>>> diff --git a/phoenix-server-client/pom.xml
>>> b/phoenix-server-client/pom.xml
>>> index e9f4bdb..f54f157 100644
>>> --- a/phoenix-server-client/pom.xml
>>> +++ b/phoenix-server-client/pom.xml
>>> @@ -54,8 +54,8 @@
>>>
>>>    <dependencies>
>>>      <dependency>
>>> -<groupId>org.apache.calcite</groupId>
>>> -<artifactId>calcite-avatica</artifactId>
>>> +<groupId>org.apache.calcite.avatica</groupId>
>>> +<artifactId>avatica-parent</artifactId>
>>>      </dependency>
>>>      <dependency>
>>>        <!-- a dependency for the thin-client uberjar -->
>>> diff --git a/phoenix-server/pom.xml b/phoenix-server/pom.xml
>>> index 9eb0e33..7563762 100644
>>> --- a/phoenix-server/pom.xml
>>> +++ b/phoenix-server/pom.xml
>>> @@ -66,13 +66,8 @@
>>>        <artifactId>phoenix-server-client</artifactId>
>>>      </dependency>
>>>      <dependency>
>>> -<groupId>org.apache.calcite</groupId>
>>> -<artifactId>calcite-avatica</artifactId>
>>> -<version>${calcite.version}</version>
>>> -</dependency>
>>> -<dependency>
>>> -<groupId>org.apache.calcite</groupId>
>>> -<artifactId>calcite-avatica-server</artifactId>
>>> +<groupId>org.apache.calcite.avatica</groupId>
>>> +<artifactId>avatica-parent</artifactId>
>>>        <version>${calcite.version}</version>
>>>      </dependency>
>>>      <dependency>
>>> diff --git a/pom.xml b/pom.xml
>>> index 1c7dc53..4fdc210 100644
>>> --- a/pom.xml
>>> +++ b/pom.xml
>>> @@ -51,6 +51,10 @@
>>>        </snapshots>
>>>      </repository>
>>>      <repository>
>>> +<id>apache calcite avatica RC</id>
>>> +<url>
>>> https://repository.apache.org/content/repositories/orgapachecalcite-1014
>>> </url>
>>> +</repository>
>>> +<repository>
>>>        <id>sonatype-nexus-snapshots</id>
>>>        <name>Sonatype Nexus Snapshots</name>
>>>        <url>https://oss.sonatype.org/content/repositories/snapshots
>>> </url>
>>> @@ -110,7 +114,7 @@
>>>      <!-- Do not change jodatime.version until HBASE-15199 is fixed -->
>>>      <jodatime.version>1.6</jodatime.version>
>>>      <joni.version>2.1.2</joni.version>
>>> -<calcite.version>1.6.0</calcite.version>
>>> +<calcite.version>1.7.0</calcite.version>
>>>      <jettyVersion>8.1.7.v20120910</jettyVersion>
>>>      <tephra.version>0.7.0</tephra.version>
>>>      <spark.version>1.5.2</spark.version>
>>> @@ -590,13 +594,8 @@
>>>          </exclusions>
>>>        </dependency>
>>>        <dependency>
>>> -<groupId>org.apache.calcite</groupId>
>>> -<artifactId>calcite-avatica</artifactId>
>>> -<version>${calcite.version}</version>
>>> -</dependency>
>>> -<dependency>
>>> -<groupId>org.apache.calcite</groupId>
>>> -<artifactId>calcite-avatica-server</artifactId>
>>> +<groupId>org.apache.calcite.avatica</groupId>
>>> +<artifactId>avatica-parent</artifactId>
>>>          <version>${calcite.version}</version>
>>>        </dependency>
>>>
>>>
>>>
>>> On Fri, Mar 11, 2016 at 10:40 AM, Julian Hyde<jhyde@apache.org>  wrote:
>>>
>>> Note that we changed the group id from “org.apache.calcite” to
>>>> “org.apache.calcite.avatica”. I don’t know whether that has sufficient
>>>> prominence in the release notes but it could definitely trip you up when
>>>> upgrading.
>>>>
>>>> On Mar 11, 2016, at 10:39 AM, James Taylor<jamestaylor@apache.org>
>>>>>
>>>> wrote:
>>>>
>>>>> To verify the release, I'm trying to change Phoenix so it only depends
>>>>> on
>>>>> avatica (as that's the only part of Calcite we're currently relying on
>>>>> in
>>>>> our releases), but my maven knowledge is lacking.
>>>>>
>>>>> Any chance you could help with this, Nick?
>>>>>
>>>>> On Thu, Mar 10, 2016 at 2:25 PM, Josh Elser<josh.elser@gmail.com>
>>>>>
>>>> wrote:
>>>>
>>>>> Argh! Yes, thanks for clarifying. Sloppy copy-paste.
>>>>>>
>>>>>>
>>>>>> Julian Hyde wrote:
>>>>>>
>>>>>> The artifacts to be voted on are located here:
>>>>>>>
>>>>>>>>
>>>>>>>>
>>>> https://dist.apache.org/repos/dist/dev/calcite/apache-calcite-avatica-1.7.0-rc0/
>>>>
>>>>> Clearly you meant
>>>>>>>
>>>>>>>
>>>>>>>
>>>> https://dist.apache.org/repos/dist/dev/calcite/apache-calcite-avatica-1.7.0-rc1/
>>>>
>>>>> .
>>>>>>>
>>>>>>>
>>>>
>>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message