wombatu-kun commented on code in PR #12695:
URL: https://github.com/apache/hudi/pull/12695#discussion_r1940372971


##########
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:
   @yihua  Also, may be it's better to completely remove HDFSParquetImporter 
with hudi-cli IMPORT/UPSERT-commands, 'coz it really looks like a dead code.



-- 
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]

Reply via email to