[ 
https://issues.apache.org/jira/browse/FLINK-1967?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14625186#comment-14625186
 ] 

ASF GitHub Bot commented on FLINK-1967:
---------------------------------------

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

    https://github.com/apache/flink/pull/906#discussion_r34500553
  
    --- Diff: 
flink-staging/flink-streaming/flink-streaming-core/src/main/java/org/apache/flink/streaming/runtime/streamrecord/StreamRecordSerializer.java
 ---
    @@ -20,26 +20,35 @@
     
     import java.io.IOException;
     
    -import org.apache.flink.api.common.ExecutionConfig;
    -import org.apache.flink.api.common.typeinfo.TypeInformation;
    +import com.google.common.base.Preconditions;
     import org.apache.flink.api.common.typeutils.TypeSerializer;
     import org.apache.flink.core.memory.DataInputView;
     import org.apache.flink.core.memory.DataOutputView;
    +import org.apache.flink.streaming.api.watermark.Watermark;
    +
    +/**
    + * Serializer for {@link StreamRecord} and {@link Watermark}. This does 
not behave like a normal
    + * {@link TypeSerializer}, instead, this is only used at the
    + * {@link org.apache.flink.streaming.runtime.tasks.StreamTask} level for 
transmitting
    + * {@link StreamRecord StreamRecords} and {@link Watermark Watermarks}. 
This serializer
    + * can handle both of them, therefore it returns {@link Object} the result 
has
    + * to be cast to the correct type.
    + *
    + * @param <T> The type of value in the {@link StreamRecord}
    + */
    +public final class StreamRecordSerializer<T> extends 
TypeSerializer<Object> {
     
    -public final class StreamRecordSerializer<T> extends 
TypeSerializer<StreamRecord<T>> {
    +   private final long IS_WATERMARK = Long.MIN_VALUE;
    --- End diff --
    
    Why isn't this field a boolean? Just wondering :)


> Introduce (Event)time in Streaming
> ----------------------------------
>
>                 Key: FLINK-1967
>                 URL: https://issues.apache.org/jira/browse/FLINK-1967
>             Project: Flink
>          Issue Type: Improvement
>            Reporter: Aljoscha Krettek
>            Assignee: Aljoscha Krettek
>
> This requires introducing a timestamp in streaming record and a change in the 
> sources to add timestamps to records. This will also introduce punctuations 
> (or low watermarks) to allow windows to work correctly on unordered, 
> timestamped input data. In the process of this, the windowing subsystem also 
> needs to be adapted to use the punctuations. Furthermore, all operators need 
> to be made aware of punctuations and correctly forward them. Then, a new 
> operator must be introduced to to allow modification of timestamps.



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

Reply via email to