xushiyan commented on code in PR #9041:
URL: https://github.com/apache/hudi/pull/9041#discussion_r1246658153
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/metadata/HoodieBackedTableMetadataWriter.java:
##########
@@ -459,11 +459,6 @@ private Pair<Integer, HoodieData<HoodieRecord>>
initializeRecordIndexPartition()
final HoodieMetadataFileSystemView fsView = new
HoodieMetadataFileSystemView(dataMetaClient,
dataMetaClient.getActiveTimeline(), metadata);
- // MOR tables are not supported
- if (!dataMetaClient.getTableType().equals(HoodieTableType.COPY_ON_WRITE)) {
- throw new HoodieMetadataException("Only COW tables are supported with
record index");
- }
-
Review Comment:
this change will be included in functional test PR (which should be merged
first). i include it here for CI to pass. when merging, this diff should be
auto-resolved.
--
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: commits-unsubscr...@hudi.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
