codope commented on code in PR #10860:
URL: https://github.com/apache/hudi/pull/10860#discussion_r1525665975
##########
hudi-common/src/main/java/org/apache/hudi/common/table/HoodieTableConfig.java:
##########
@@ -737,11 +738,12 @@ public boolean isMetadataTableAvailable() {
/**
* Checks if metadata table is enabled and the specified partition has been
initialized.
*
- * @param partition The partition to check
+ * @param metadataPartitionType The metadata table partition type to check
* @returns true if the specific partition has been initialized, else
returns false.
*/
- public boolean isMetadataPartitionAvailable(MetadataPartitionType partition)
{
- return getMetadataPartitions().contains(partition.getPartitionPath());
+ public boolean isMetadataPartitionAvailable(MetadataPartitionType
metadataPartitionType) {
Review Comment:
i changed the logic a bit. We will still find func index in the mdt
partitions to initialize, but then when we actually go about initializing we
will check based on full partition path - see `initializeFromFileSystem` and
`getFunctionalIndexPartitionsToInit` in `HoodieBackedTableMetadataWriter`
--
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]
