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

Vihang Karajgaonkar edited comment on HIVE-21586 at 4/30/19 1:30 AM:
---------------------------------------------------------------------

Looks like I missed moving about {{ColumnStatistics}} and {{ReplLastIdInfo}} 
Reopening this jira. Will attach a addendum soon


was (Author: vihangk1):
Looks like I missed moving about {{ColumnStatistics}} Reopening this jira. Will 
attach a addendum soon

> Thrift generated cpp files for metastore do not compile
> -------------------------------------------------------
>
>                 Key: HIVE-21586
>                 URL: https://issues.apache.org/jira/browse/HIVE-21586
>             Project: Hive
>          Issue Type: Bug
>          Components: Standalone Metastore
>    Affects Versions: 3.1.0
>            Reporter: Vihang Karajgaonkar
>            Assignee: Vihang Karajgaonkar
>            Priority: Major
>             Fix For: 4.0.0, 3.2.0
>
>         Attachments: HIVE-21586.01.patch, HIVE-21586.02.patch, 
> HIVE-21586.03.patch, HIVE-21586.04.branch-3.patch, HIVE-21586.04.patch, 
> HIVE-21586.05.patch, HIVE-21586.nothrift.patch
>
>
> The way some structs like CreationMetadata, CompactionInfo, ColumnStatistics 
> are defined in hive_metastore.thrift is that these structs are used before 
> they are defined. While this works for the java code which is generated, it 
> does not work for the generated cpp code since Thrift does not use 
> pointer/references to the forward declared classes.
> The easy fix for this would be to reorder the struct definitions in the 
> hive_metastore.thrift so that they are always defined before they are used.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to