[ 
https://issues.apache.org/jira/browse/PHOENIX-7080?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17777138#comment-17777138
 ] 

ASF GitHub Bot commented on PHOENIX-7080:
-----------------------------------------

NihalJain commented on code in PR #8:
URL: https://github.com/apache/phoenix-thirdparty/pull/8#discussion_r1365162606


##########
pom.xml:
##########
@@ -68,7 +68,7 @@
 
   <properties>
     <rename.offset>org.apache.phoenix.thirdparty</rename.offset>
-    <guava.version>32.1.1-android</guava.version>
+    <guava.version>32.1.2-jre</guava.version>

Review Comment:
   Updated!





> Switch phoenix-thirdparty to guava-jre from guava-android
> ---------------------------------------------------------
>
>                 Key: PHOENIX-7080
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-7080
>             Project: Phoenix
>          Issue Type: Improvement
>          Components: thirdparty
>    Affects Versions: thirdparty-2.0.0
>            Reporter: Nihal Jain
>            Assignee: Nihal Jain
>            Priority: Major
>
> As discussed in mail thread, here, we want to switch to guava-jre from 
> guava-android for phoenix thirdparty
> Some more context on why we want to do this (Copied from PHOENIX-6817) is as 
> below:
> {quote}We chose to include the -android variant of Guava, to ensure 
> compatibiity with Java 7, which was required by the 4.x branch.
> Now that the 4.x branch is EOL, we can switch to the more standard -jre 
> version.
> {quote}
> CC: [~stoty] [~rajeshbabu] [~vjasani] 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to