ad1happy2go commented on PR #19248: URL: https://github.com/apache/hudi/pull/19248#issuecomment-4968196471
Pushed `3bc8210` — hardened per the review discussion. In addition to normalizing `META_SYNC_BASE_PATH` in the `HoodieSyncConfig` constructor, the table `LOCATION` is now also canonicalized **at the `setLocation` boundary** (`HMSDDLExecutor#createTable` and `HoodieHiveSyncClient#updateLastCommitTimeSynced`) via `new Path(...).toString()`. This guarantees the metastore never receives a `//` LOCATION even if the base path reaches the executor un-normalized (e.g. a config value that slipped through raw). Partition locations already normalize through `FSUtils#constructAbsolutePath`, so they're unchanged. Added `TestHMSDDLExecutor` asserting a `gs://.../tbl//` base path yields a clean `gs://.../tbl` table LOCATION. -- 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]
