This is an automated email from the ASF dual-hosted git repository.
danny0405 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hudi.git
The following commit(s) were added to refs/heads/master by this push:
new e71e6cc3f07c refactor(client): remove unused FlinkHoodieIndex and
JavaHoodieIndex (#19975)
e71e6cc3f07c is described below
commit e71e6cc3f07c753b565fcf7739e14c12850f3733
Author: voonhous <[email protected]>
AuthorDate: Thu Sep 17 09:39:29 2026 +0800
refactor(client): remove unused FlinkHoodieIndex and JavaHoodieIndex
(#19975)
Both are identical abstract HoodieIndex shims over List-shaped
records. Nothing has extended either since #3778 moved the engine
indexes into hudi-client-common, and nothing else references them.
---
.../org/apache/hudi/index/FlinkHoodieIndex.java | 74 ----------------------
.../org/apache/hudi/index/JavaHoodieIndex.java | 71 ---------------------
2 files changed, 145 deletions(-)
diff --git
a/hudi-client/hudi-flink-client/src/main/java/org/apache/hudi/index/FlinkHoodieIndex.java
b/hudi-client/hudi-flink-client/src/main/java/org/apache/hudi/index/FlinkHoodieIndex.java
deleted file mode 100644
index f43130234f8d..000000000000
---
a/hudi-client/hudi-flink-client/src/main/java/org/apache/hudi/index/FlinkHoodieIndex.java
+++ /dev/null
@@ -1,74 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements. See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership. The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied. See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-
-package org.apache.hudi.index;
-
-import org.apache.hudi.ApiMaturityLevel;
-import org.apache.hudi.PublicAPIMethod;
-import org.apache.hudi.client.WriteStatus;
-import org.apache.hudi.common.data.HoodieData;
-import org.apache.hudi.common.data.HoodieListData;
-import org.apache.hudi.common.engine.HoodieEngineContext;
-import org.apache.hudi.common.model.HoodieRecord;
-import org.apache.hudi.config.HoodieWriteConfig;
-import org.apache.hudi.exception.HoodieIndexException;
-import org.apache.hudi.table.HoodieTable;
-
-import java.util.List;
-import java.util.stream.Collectors;
-
-/**
- * Base flink implementation of {@link HoodieIndex}.
- */
-public abstract class FlinkHoodieIndex<T> extends
HoodieIndex<List<HoodieRecord<T>>, List<WriteStatus>> {
- protected FlinkHoodieIndex(HoodieWriteConfig config) {
- super(config);
- }
-
- @Override
- @Deprecated
- @PublicAPIMethod(maturity = ApiMaturityLevel.DEPRECATED)
- public abstract List<WriteStatus> updateLocation(List<WriteStatus>
writeStatuses,
- HoodieEngineContext context,
- HoodieTable hoodieTable)
throws HoodieIndexException;
-
- @Override
- @Deprecated
- @PublicAPIMethod(maturity = ApiMaturityLevel.DEPRECATED)
- public abstract List<HoodieRecord<T>> tagLocation(List<HoodieRecord<T>>
records,
- HoodieEngineContext
context,
- HoodieTable hoodieTable)
throws HoodieIndexException;
-
- @Override
- @PublicAPIMethod(maturity = ApiMaturityLevel.EVOLVING)
- public <R> HoodieData<HoodieRecord<R>> tagLocation(
- HoodieData<HoodieRecord<R>> records, HoodieEngineContext context,
- HoodieTable hoodieTable) throws HoodieIndexException {
- List<HoodieRecord<T>> hoodieRecords = tagLocation(records.map(record ->
(HoodieRecord<T>) record).collectAsList(), context, hoodieTable);
- return HoodieListData.eager(hoodieRecords.stream().map(r ->
(HoodieRecord<R>) r).collect(Collectors.toList()));
- }
-
- @Override
- @PublicAPIMethod(maturity = ApiMaturityLevel.EVOLVING)
- public HoodieData<WriteStatus> updateLocation(
- HoodieData<WriteStatus> writeStatuses, HoodieEngineContext context,
- HoodieTable hoodieTable) throws HoodieIndexException {
- return HoodieListData.eager(updateLocation(writeStatuses.collectAsList(),
context, hoodieTable));
- }
-}
diff --git
a/hudi-client/hudi-java-client/src/main/java/org/apache/hudi/index/JavaHoodieIndex.java
b/hudi-client/hudi-java-client/src/main/java/org/apache/hudi/index/JavaHoodieIndex.java
deleted file mode 100644
index f8efb141fdfe..000000000000
---
a/hudi-client/hudi-java-client/src/main/java/org/apache/hudi/index/JavaHoodieIndex.java
+++ /dev/null
@@ -1,71 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one
- * or more contributor license agreements. See the NOTICE file
- * distributed with this work for additional information
- * regarding copyright ownership. The ASF licenses this file
- * to you under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance
- * with the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing,
- * software distributed under the License is distributed on an
- * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- * KIND, either express or implied. See the License for the
- * specific language governing permissions and limitations
- * under the License.
- */
-
-package org.apache.hudi.index;
-
-import org.apache.hudi.ApiMaturityLevel;
-import org.apache.hudi.PublicAPIMethod;
-import org.apache.hudi.client.WriteStatus;
-import org.apache.hudi.common.data.HoodieData;
-import org.apache.hudi.common.data.HoodieListData;
-import org.apache.hudi.common.engine.HoodieEngineContext;
-import org.apache.hudi.common.model.HoodieRecord;
-import org.apache.hudi.config.HoodieWriteConfig;
-import org.apache.hudi.exception.HoodieIndexException;
-import org.apache.hudi.table.HoodieTable;
-
-import java.util.List;
-import java.util.stream.Collectors;
-
-public abstract class JavaHoodieIndex<T> extends
HoodieIndex<List<HoodieRecord<T>>, List<WriteStatus>> {
- protected JavaHoodieIndex(HoodieWriteConfig config) {
- super(config);
- }
-
- @Override
- @Deprecated
- @PublicAPIMethod(maturity = ApiMaturityLevel.DEPRECATED)
- public abstract List<WriteStatus> updateLocation(List<WriteStatus>
writeStatuses,
- HoodieEngineContext context,
- HoodieTable hoodieTable)
throws HoodieIndexException;
-
- @Override
- @Deprecated
- @PublicAPIMethod(maturity = ApiMaturityLevel.DEPRECATED)
- public abstract List<HoodieRecord<T>> tagLocation(List<HoodieRecord<T>>
records,
- HoodieEngineContext
context,
- HoodieTable hoodieTable)
throws HoodieIndexException;
-
- @Override
- @PublicAPIMethod(maturity = ApiMaturityLevel.EVOLVING)
- public <R> HoodieData<HoodieRecord<R>> tagLocation(
- HoodieData<HoodieRecord<R>> records, HoodieEngineContext context,
- HoodieTable hoodieTable) throws HoodieIndexException {
- List<HoodieRecord<T>> hoodieRecords = tagLocation(records.map(record ->
(HoodieRecord<T>) record).collectAsList(), context, hoodieTable);
- return HoodieListData.eager(hoodieRecords.stream().map(r ->
(HoodieRecord<R>) r).collect(Collectors.toList()));
- }
-
- @Override
- @PublicAPIMethod(maturity = ApiMaturityLevel.EVOLVING)
- public HoodieData<WriteStatus> updateLocation(
- HoodieData<WriteStatus> writeStatuses, HoodieEngineContext context,
- HoodieTable hoodieTable) throws HoodieIndexException {
- return HoodieListData.eager(updateLocation(writeStatuses.collectAsList(),
context, hoodieTable));
- }
-}
