myg821561935 opened a new pull request, #6628: URL: https://github.com/apache/hive/pull/6628
### What changed Partition creation goes through the metastore, which assigns server-owned fields such as `createTime`. The load path was discarding the `Partition` returned by HMS and continued with the client-side object, whose `createTime` was still zero. A later alter-partition call could then write that zero back to the metastore. This change: * keeps the HMS-returned partition in both single-partition and batch dynamic-partition load paths; * requests results from the batch `add_partitions` call and updates the returned partition map with those results; * preserves the existing partition create time in single, rename, and batch alter paths when the incoming value is unset or non-positive; and * adds QL and metastore regression coverage for both creation and alter paths. The metastore guard does not override an explicitly supplied positive create time, and there is no public API change. ### Validation * `TestHiveLoadPartitionKeepExisting`: 5 tests passed, including the new static and dynamic partition cases. * `TestAlterPartitions`: 154 tests passed, including the new single and batch alter cases. * The related 21-module Maven reactor completed successfully with JDK 21. JIRA: https://issues.apache.org/jira/browse/HIVE-28757 -- 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]
