flyrain commented on code in PR #4870: URL: https://github.com/apache/hive/pull/4870#discussion_r1389948114
########## metastore/if/hive_metastore.thrift: ########## @@ -1550,4 +1562,4 @@ const string TABLE_TRANSACTIONAL_PROPERTIES = "transactional_properties", // Keys for alter table environment context parameters const string EXPECTED_PARAMETER_KEY = "expected_parameter_key", -const string EXPECTED_PARAMETER_VALUE = "expected_parameter_value", \ No newline at end of file +const string EXPECTED_PARAMETER_VALUE = "expected_parameter_value", Review Comment: I'm OK with either style. It's funny that google java formatter requires an empty line at every file. -- 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]
