SteNicholas commented on code in PR #826:
URL: https://github.com/apache/incubator-paimon/pull/826#discussion_r1156842861
##########
paimon-common/src/main/java/org/apache/paimon/utils/HadoopUtils.java:
##########
@@ -38,26 +38,8 @@ public class HadoopUtils {
public static final String HADOOP_HOME_ENV = "HADOOP_HOME";
public static final String HADOOP_CONF_ENV = "HADOOP_CONF_DIR";
- /**
- * Path to hdfs-default.xml file.
- *
- * @deprecated Use environment variable HADOOP_CONF_DIR instead.
- */
- @Deprecated public static final String HDFS_DEFAULT_CONFIG =
"fs.hdfs.hdfsdefault";
-
- /**
- * Path to hdfs-site.xml file.
- *
- * @deprecated Use environment variable HADOOP_CONF_DIR instead.
- */
- @Deprecated public static final String HDFS_SITE_CONFIG =
"fs.hdfs.hdfssite";
-
- /**
- * Path to Hadoop configuration.
- *
- * @deprecated Use environment variable HADOOP_CONF_DIR instead.
- */
- @Deprecated public static final String PATH_HADOOP_CONFIG =
"fs.hdfs.hadoopconf";
+ /** Path to Hadoop configuration. */
+ public static final String PATH_HADOOP_CONFIG = "fs.hdfs.hadoopconf";
Review Comment:
BTW, this change should be recorded in release note.
--
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: issues-unsubscr...@paimon.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
