Github user uce commented on a diff in the pull request:

    https://github.com/apache/incubator-flink/pull/239#discussion_r20989360
  
    --- Diff: 
flink-core/src/main/java/org/apache/flink/api/common/typeutils/base/array/BooleanPrimitiveArraySerializer.java
 ---
    @@ -45,10 +43,16 @@ public boolean isImmutableType() {
        public boolean isStateful() {
                return false;
        }
    -   
    +
    +   @Override
    +   public boolean canCreateInstance() {
    +           return false;
    +   }
    +
        @Override
        public boolean[] createInstance() {
    -           return EMPTY;
    +           throw new 
UnsupportedOperationException("BooleanPrimitiveArraySerializer cannot create 
an" +
    --- End diff --
    
    I think most of the project code is using 120 chars for line breaks, so you 
don't need the line break here if you don't want to


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to