You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "Fabian Hueske (JIRA)" <ji...@apache.org> on 2017/07/28 13:21:00 UTC
[jira] [Updated] (FLINK-7291) Incorrect return value of getArity()
in ListTypeInfo and MapTypeInfo
[ https://issues.apache.org/jira/browse/FLINK-7291?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Fabian Hueske updated FLINK-7291:
---------------------------------
Description:
The implementation of {{getArity()}} in {{ListTypeInfo}} and {{MapTypeInfo}} returns 0 but should return 1.
The implementation of {{MapTypeInfo.getTotalFields()}} is also incorrect and must return 1 instead of 2.
The JavaDocs of {{TypeInformation.getArity()}} and {{TypeInformation.getTotalFields()}} should be extended as well to avoid future confusion about this method. The method returns the arity for current type. For atomic types, this is 1 for composite types, the number of non-nested fields (or number of fields on the first nesting-level).
was:
The implementation of {{getArity()}} in {{ListTypeInfo}} and {{MapTypeInfo}} returns 0 but should return 1.
The JavaDocs of {{TypeInformation.getArity()}} should be extended as well to avoid future confusion about this method. The method returns the arity for current type. For atomic types, this is 1 for composite types, the number of non-nested fields (or number of fields on the first nesting-level).
> Incorrect return value of getArity() in ListTypeInfo and MapTypeInfo
> --------------------------------------------------------------------
>
> Key: FLINK-7291
> URL: https://issues.apache.org/jira/browse/FLINK-7291
> Project: Flink
> Issue Type: Bug
> Components: Core
> Affects Versions: 1.3.1, 1.4.0
> Reporter: Fabian Hueske
>
> The implementation of {{getArity()}} in {{ListTypeInfo}} and {{MapTypeInfo}} returns 0 but should return 1.
> The implementation of {{MapTypeInfo.getTotalFields()}} is also incorrect and must return 1 instead of 2.
> The JavaDocs of {{TypeInformation.getArity()}} and {{TypeInformation.getTotalFields()}} should be extended as well to avoid future confusion about this method. The method returns the arity for current type. For atomic types, this is 1 for composite types, the number of non-nested fields (or number of fields on the first nesting-level).
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)