weibozhao commented on a change in pull request #24:
URL: https://github.com/apache/flink-ml/pull/24#discussion_r756819327
##########
File path:
flink-ml-lib/src/main/java/org/apache/flink/ml/common/param/HasFeatureColsDefaultAsNull.java
##########
@@ -0,0 +1,28 @@
+package org.apache.flink.ml.common.param;
+
+import org.apache.flink.ml.param.Param;
+import org.apache.flink.ml.param.ParamValidators;
+import org.apache.flink.ml.param.WithParams;
+
+/** Params of the names of the feature columns used for training in the input
table. */
+public interface HasFeatureColsDefaultAsNull<T> extends WithParams<T> {
+ /**
+ * @cn-name 特征列名数组
+ * @cn 特征列名数组，默认全选
+ */
+ Param<String[]> FEATURE_COLS =
Review comment:
I think we need discuss the format of features input for the algorithm.
A dense vector or a string format vector or some numerical columns?
--
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...@flink.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
