[ 
https://issues.apache.org/jira/browse/FLINK-3866?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Tatu Saloranta updated FLINK-3866:
----------------------------------
    Description: 
Looking at default `TypeSerializer` instances I noticed what looks like a minor 
flaw, unless I am missing something.
Whereas all other array serializers indicate that type is not immutable (since 
in Java, arrays are not immutable), `StringArraySerializer` has:

```
        @Override
        public boolean isImmutableType() {
                return true;
        }
```

and I think it should instead return `false`.




  was:
Jackson version in use (2.4.2) is rather old (and not even the latest patch 
from minor version), so it'd be make sense to upgrade to bit newer. Latest 
would be 2.7.4, but at first I propose going to 2.5.5.

All tests pass, but if there are issues I'd be happy to help; I'm author of 
Jackson project.



> StringArraySerializer claims type is immutable; shouldn't
> ---------------------------------------------------------
>
>                 Key: FLINK-3866
>                 URL: https://issues.apache.org/jira/browse/FLINK-3866
>             Project: Flink
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 1.0.3
>            Reporter: Tatu Saloranta
>            Priority: Minor
>
> Looking at default `TypeSerializer` instances I noticed what looks like a 
> minor flaw, unless I am missing something.
> Whereas all other array serializers indicate that type is not immutable 
> (since in Java, arrays are not immutable), `StringArraySerializer` has:
> ```
>       @Override
>       public boolean isImmutableType() {
>               return true;
>       }
> ```
> and I think it should instead return `false`.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to