Takch02 commented on PR #4137: URL: https://github.com/apache/amoro/pull/4137#issuecomment-4252498093
@czy006 @xxubai Non-partitioned Hive tables support only a single directory as their location. Previously, partial rewrites or overwrites that moved files to a new directory were allowed even when existing files remained in the old location, causing those files to become invisible to Hive. Added checkNonPartitionedHiveLocationChange() in UpdateHiveFiles to validate that all files in the current Hive location are included in the delete set before allowing a location change. The check uses the Iceberg snapshot rather than the filesystem to avoid false positives from files written before commit. Also fixed non-partitioned test cases to TestRewriteFiles and TestOverwriteFiles to verify that CannotAlterHiveLocationException is thrown in these scenarios, consistent with the existing behavior for partitioned tables. -- 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]
