wecharyu commented on PR #4744: URL: https://github.com/apache/hive/pull/4744#issuecomment-1869028375
> Currently DirectSqlUpdateStat is used only in ETL flow, can we use directSql stats update in HiveAlterHandler(rename table/partition/col, drop col) as well ? Yes, here we can make some improvement, for example renaming table/partition does not need update column statistic. > Please check whether all this columns can be removed from hive_metastore.thrift ColumnStatisticsDesc We can not remove these columns in `ColumnStatisticsDecs`, because: 1. it's the API data struct, changing the struct would cause API incompatible. 2. these columns are meaningful to describe the table or partition metadata. -- 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]
