You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hive.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2022/09/10 14:29:00 UTC
[jira] [Work logged] (HIVE-26468) Cleanup dependency to FlatBuffers
[ https://issues.apache.org/jira/browse/HIVE-26468?focusedWorklogId=807659&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-807659 ]
ASF GitHub Bot logged work on HIVE-26468:
-----------------------------------------
Author: ASF GitHub Bot
Created on: 10/Sep/22 14:28
Start Date: 10/Sep/22 14:28
Worklog Time Spent: 10m
Work Description: slachiewicz commented on code in PR #3517:
URL: https://github.com/apache/hive/pull/3517#discussion_r967660927
##########
serde/pom.xml:
##########
@@ -81,7 +81,7 @@
<version>${hppc.version}</version>
</dependency>
<dependency>
- <groupId>com.github.davidmoten</groupId>
+ <groupId>com.google.flatbuffers</groupId>
Review Comment:
@ayushtkn anything more I can help with review?
Issue Time Tracking
-------------------
Worklog Id: (was: 807659)
Time Spent: 40m (was: 0.5h)
> Cleanup dependency to FlatBuffers
> ---------------------------------
>
> Key: HIVE-26468
> URL: https://issues.apache.org/jira/browse/HIVE-26468
> Project: Hive
> Issue Type: Bug
> Reporter: Sylwester Lachiewicz
> Priority: Minor
> Labels: pull-request-available
> Time Spent: 40m
> Remaining Estimate: 0h
>
> Remove issue with overlapping dependencies to FlatBuffers.
> Warning inside shading step for hive-jdbc, hive-beeline shows that we have wring dependencies. Similar issue exists inside hive-serde.
>
> Simply switch to Google FlatBuffers artefact
>
> [WARNING] flatbuffers-java-1.9.0.jar, flatbuffers-java-1.6.0.1.jar define 5 overlapping classes:
> [WARNING] - com.google.flatbuffers.Struct
> [WARNING] - com.google.flatbuffers.Table$1
> [WARNING] - com.google.flatbuffers.Constants
> [WARNING] - com.google.flatbuffers.Table
> [WARNING] - com.google.flatbuffers.FlatBufferBuilder
--
This message was sent by Atlassian Jira
(v8.20.10#820010)