[HIVE-21586] Thrift generated cpp files for metastore do not compile

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.1.0
• Fix Version/s: 3.2.0, 4.0.0-alpha-1
• Component/s: Standalone Metastore
• Assignee: Vihang Karajgaonkar
• Reporter: Vihang Karajgaonkar
• Created: 05/Apr/19 17:53
• Updated: 17/Nov/22 08:50
• Resolved: 01/May/19 00:30

Description

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.

Issue Links

• relates to: THRIFT-4846 C++ generator should topologically sort struct definitions and error on cycles