prashantwason commented on PR #17775: URL: https://github.com/apache/hudi/pull/17775#issuecomment-3803534505
@nsivabalan Thanks for the review! I've addressed your comments: **Tests Added:** Created `TestSparkHoodieTableFileIndexHmsListing.scala` with tests that verify: - Config wiring is correct (HMS listing only used when enabled) - When config is enabled and MDT is disabled, partitions come from catalog - When MDT is available, HMS listing is skipped (prefers faster MDT-based listing) - Partition prefix filtering works correctly - End-to-end partition pruning works with HMS listing **MDT Check Added:** Updated `getMatchingPartitionPathsFromCatalog` to also check `isMetadataTableAvailable`. The HMS listing path is now only used when the config is enabled AND MDT is not available. -- 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]
