[
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)