yihua commented on code in PR #12695:
URL: https://github.com/apache/hudi/pull/12695#discussion_r1938884960
##########
hudi-cli/src/main/java/org/apache/hudi/cli/utils/HDFSParquetImporter.java:
##########
@@ -58,17 +54,14 @@
import java.time.ZoneId;
import java.time.format.DateTimeFormatter;
import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collections;
import java.util.List;
import scala.Tuple2;
/**
- * Loads data from Parquet Sources.
- *
- * @see HoodieStreamer
- * @deprecated This utility is deprecated in 0.10.0 and will be removed in
0.11.0. Use {@link HoodieStreamer.Config#runBootstrap} instead.
+ * Loads data from Parquet Sources. This class is moved to hudi-cli from
hudi-utilities.
+ * This feature, as separate utility, was deprecated in 0.10.0 and should be
removed in 0.11.0.
+ * Now it is used only by hudi-cli SparkMain IMPORT and UPSERT commands (for
compatibility support of hudi-cli API).
Review Comment:
Do we have test coverage on this class? Could you confirm which test covers
the logic and if the CI actually runs the test?
--
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]
