[
https://issues.apache.org/jira/browse/APEXMALHAR-2111?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15325224#comment-15325224
]
ASF GitHub Bot commented on APEXMALHAR-2111:
--------------------------------------------
Github user chinmaykolhatkar commented on a diff in the pull request:
https://github.com/apache/apex-malhar/pull/311#discussion_r66675300
--- Diff:
library/src/main/java/com/datatorrent/lib/projection/ProjectionOperator.java ---
@@ -308,5 +303,45 @@ private void handleProjection(Object t)
}
}
+ /**
+ * set selectFields, a list of fields to be selected from incoming POJO
+ *
+ * @param selectFields List of fields from POJO to be selected
+ */
+ public void setSelectFields(List<String> selectFields)
+ {
+ this.selectFields = selectFields;
+ }
+
+ /**
+ * get selectFields, a list of fields to be selected from incoming POJO
+ *
+ * @return selectFields list of fields from POJO to be selected
+ */
+ public List<String> getSelectFields()
+ {
+ return selectFields;
+ }
+
+ /**
+ * set dropFields, a list of fields to be dropped from incoming POJO
+ *
+ * @param dropFields List of fields from POJO to be selected
+ */
+ public void setDropFields(List<String> dropFields)
--- End diff --
Same as selectFields. Please add @useSchema and @description metatag.
> Projection Operator config params shall use List instead of comma-separated
> field names
> ---------------------------------------------------------------------------------------
>
> Key: APEXMALHAR-2111
> URL: https://issues.apache.org/jira/browse/APEXMALHAR-2111
> Project: Apache Apex Malhar
> Issue Type: Improvement
> Reporter: Pradeep A. Dalvi
> Assignee: Pradeep A. Dalvi
>
> Projection Operator accepts 2 config params: selectFields & dropFields.
> Currently both these parameters accept values in comma-separated field names
> format. However this is not inline with other operators do to accept multiple
> values, for which they use List instead.
> So this proposal is to make change in Projection Operator to accept config
> params as List instead of comma-separated string. Hence selectFields &
> dropFields shall accept List going forward.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
