[
https://issues.apache.org/jira/browse/FLINK-3702?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15432430#comment-15432430
]
ASF GitHub Bot commented on FLINK-3702:
---------------------------------------
Github user twalthr commented on a diff in the pull request:
https://github.com/apache/flink/pull/2094#discussion_r75826683
--- Diff:
flink-core/src/main/java/org/apache/flink/api/common/typeinfo/BasicArrayTypeInfo.java
---
@@ -121,6 +122,18 @@ public boolean isKeyType() {
}
@Override
+ @PublicEvolving
+ public <F> FieldAccessor<T, F> getFieldAccessor(int pos,
ExecutionConfig config) {
+ return new FieldAccessor.ArrayFieldAccessor<T, F>(pos, this);
+ }
+
+ @Override
+ @PublicEvolving
+ public <F> FieldAccessor<T, F> getFieldAccessor(String field,
ExecutionConfig config) {
+ throw new InvalidFieldReferenceException("Selecting a field
from an array type is only supported by index.");
--- End diff --
We could also add an additional constructor in `ArrayFieldAccessor` that
takes a string and parses the int position there. Then we don't need to throw
exceptions. What do you think?
> DataStream API PojoFieldAccessor doesn't support nested POJOs
> -------------------------------------------------------------
>
> Key: FLINK-3702
> URL: https://issues.apache.org/jira/browse/FLINK-3702
> Project: Flink
> Issue Type: Improvement
> Components: DataStream API
> Affects Versions: 1.0.0
> Reporter: Robert Metzger
> Assignee: Gabor Gevay
>
> The {{PojoFieldAccessor}} (which is used by {{.sum(String)}} and similar
> methods) doesn't support nested POJOs right now.
> As part of FLINK-3697 I'll add a check for a nested POJO and fail with an
> exception.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
