kasakrisz opened a new pull request #2953: URL: https://github.com/apache/hive/pull/2953
### What changes were proposed in this pull request? Thrift generates cpp client with structs/classes having non-pointer fields. This blocks forward declaration. However `SourceTable` is referenced from `CreationMetadata` as a collection element type which makes thrift to generate a pointer type for `SourceTable`. To resolve the circular dependency issue move `SourceTable` definition after `Table` definition in hive_metastore.thrift. Also change the collection type `set<SourceTable>` to list<SourceTable> because set requires additional changes on the element type. ### Why are the changes needed? Fix compilation errors in cpp thrift metastore client. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Compile the cpp client. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
