You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by tzulitai <gi...@git.apache.org> on 2017/06/06 08:48:37 UTC

[GitHub] flink pull request #4079: [FLINK-6853] [DataStream] Let StreamRecordSerializ...

GitHub user tzulitai opened a pull request:

    https://github.com/apache/flink/pull/4079

    [FLINK-6853] [DataStream] Let StreamRecordSerializer be compatible with MultiplexingStreamRecordSerializer

    This PR lets `StreamRecordSerializer.ensureCompatibility` be tolerable for config snapshots taken from the legacy `MultiplexingStreamRecordSerializer`. This is required for users which originally used `MultiplexingStreamRecordSerializer` to serialize stream elements as part of their checkpointed state (e.g. FlinkCEP).


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/tzulitai/flink FLINK-6853

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/flink/pull/4079.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #4079
    
----
commit 54f5a33f8fb88d2219f06b6b874921d1d063ddca
Author: Tzu-Li (Gordon) Tai <tz...@apache.org>
Date:   2017-06-06T08:41:03Z

    [FLINK-6853] [DataStream] Let StreamRecordSerializer be compatible with MultiplexingStreamRecordSerializer
    
    This commit lets StreamRecordSerializer.ensureCompatibility be tolerable
    for config snapshots taken from the legacy
    MultiplexingStreamRecordSerializer. This is required for users which
    originally used MultiplexingStreamRecordSerializer to serialize stream
    elements as part of their checkpointed state (e.g. FlnkCEP).

----


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

[GitHub] flink pull request #4079: [FLINK-6853] [DataStream] Let StreamRecordSerializ...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/flink/pull/4079


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

[GitHub] flink issue #4079: [FLINK-6853] [DataStream] Let StreamRecordSerializer be c...

Posted by tzulitai <gi...@git.apache.org>.
Github user tzulitai commented on the issue:

    https://github.com/apache/flink/pull/4079
  
    The test results in #4073 covers this change and results are green.
    Since #4073 is not really a real blocker for 1.3.1, while this fix is to allow CEP migration from 1.1, I'll merge this independent of #4073 first. 


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