You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by nderraugh <gi...@git.apache.org> on 2016/09/30 02:40:04 UTC

[GitHub] flink pull request #2575: 'Id' is the key of the stream and not the id of th...

GitHub user nderraugh opened a pull request:

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

    'Id' is the key of the stream and not the id of the event.

    FLINK-4718 Confusing label in Parallel Streams Diagram
    
    Minor text change to clarify a mildly confusing label in a diagram.

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

    $ git pull https://github.com/nderraugh/flink master

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

    https://github.com/apache/flink/pull/2575.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 #2575
    
----
commit 479c8401f9680b4e0bf88af4c7b68033aeb54cdc
Author: Neil Derraugh <ne...@intellifylearning.com>
Date:   2016-09-29T17:53:10Z

    'Id' is the key of the stream and not the id of the event.

----


---
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 #2575: 'Id' is the key of the stream and not the id of the event...

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

    https://github.com/apache/flink/pull/2575
  
    OK. That clears it up too.  I'll fix it up in a little while.
    
    On Sep 30, 2016 6:38 AM, "Fabian Hueske" <no...@github.com> wrote:
    
    > Sorry @nderraugh <https://github.com/nderraugh>, seems like I
    > misinterpreted the figure then... :-(
    >
    > \u2014
    > You are receiving this because you were mentioned.
    > Reply to this email directly, view it on GitHub
    > <https://github.com/apache/flink/pull/2575#issuecomment-250713933>, or mute
    > the thread
    > <https://github.com/notifications/unsubscribe-auth/AA7Ojug343Zg58vAEWIdUH0NsdXJaxgeks5qvObAgaJpZM4KKrBG>
    > .
    >



---
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 #2575: 'Id' is the key of the stream and not the id of the event...

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

    https://github.com/apache/flink/pull/2575
  
    Sorry @nderraugh, seems like I misinterpreted the figure then... :-(


---
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 #2575: 'Id' is the key of the stream and not the id of the event...

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

    https://github.com/apache/flink/pull/2575
  
    I originally meant it to be the "id" or something like that.
    The confusion probably comes because "B" occurs multiple times. That is a copy and paste error, it should be a new letter.


---
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 #2575: 'Id' is the key of the stream and not the id of the event...

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

    https://github.com/apache/flink/pull/2575
  
    will reopen a clean PR for this change.


---
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 #2575: 'Id' is the key of the stream and not the id of th...

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

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


---
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.
---