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

    https://github.com/apache/flink/pull/2094#discussion_r87096436
  
    --- Diff: 
flink-streaming-java/src/main/java/org/apache/flink/streaming/api/datastream/KeyedStream.java
 ---
    @@ -337,37 +337,43 @@ public KeyedStream(DataStream<T> dataStream, 
KeySelector<T, KEY> keySelector, Ty
         * per key.
         *
         * @param positionToSum
    -    *            The position in the data point to sum
    +    *            The field position in the data points to sum. This is 
applicable to
    +    *            Tuple types, basic and primitive array types, Scala case 
classes,
    --- End diff --
    
    Not sure if we should document the behavior that we might remove soon. 
    I'd rather have this as undocumented feature and open a JIRA issue to 
either add the documentation or remove the support for arrays and `0` for 
atomic types. What do you think @mbalassi?
    
    Same applies for all other Java / ScalaDocs.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to