[GitHub] flink pull request: [FLINK-1875] Add figure explaining slots and p...

2015-04-22 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/604#discussion_r28872548 --- Diff: docs/config.md --- @@ -370,8 +370,7 @@ system, such as */tmp* in Linux systems. ### Configuring TaskManager processing slots -A

[GitHub] flink pull request: [FLINK-1875] Add figure explaining slots and p...

2015-04-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/604 --- 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 enabl

[GitHub] flink pull request: [FLINK-1875] Add figure explaining slots and p...

2015-04-21 Thread fhueske
Github user fhueske commented on the pull request: https://github.com/apache/flink/pull/604#issuecomment-94702358 Nice figure! I added a comment about a certain term in the documentation, but otherwise I think we should get it in. --- If your project is set up for it, you can reply t

[GitHub] flink pull request: [FLINK-1875] Add figure explaining slots and p...

2015-04-21 Thread fhueske
Github user fhueske commented on a diff in the pull request: https://github.com/apache/flink/pull/604#discussion_r28758925 --- Diff: docs/config.md --- @@ -370,8 +370,7 @@ system, such as */tmp* in Linux systems. ### Configuring TaskManager processing slots -A p

[GitHub] flink pull request: [FLINK-1875] Add figure explaining slots and p...

2015-04-17 Thread rmetzger
Github user rmetzger commented on the pull request: https://github.com/apache/flink/pull/604#issuecomment-93993725 The issue seems to be fixed --- 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

[GitHub] flink pull request: [FLINK-1875] Add figure explaining slots and p...

2015-04-17 Thread rmetzger
Github user rmetzger commented on the pull request: https://github.com/apache/flink/pull/604#issuecomment-93956039 It seems that the apache rat plugin has troubles verifying the license header of the svg file. I have to look into this again. --- If your project is set up for

[GitHub] flink pull request: [FLINK-1875] Add figure explaining slots and p...

2015-04-17 Thread mxm
Github user mxm commented on the pull request: https://github.com/apache/flink/pull/604#issuecomment-93950872 That's a very nice graphical explanation of parallelism. :+1: You could add another row with a job which changes parallelism in the middle of the job. For example, ca

[GitHub] flink pull request: [FLINK-1875] Add figure explaining slots and p...

2015-04-17 Thread rmetzger
Github user rmetzger commented on the pull request: https://github.com/apache/flink/pull/604#issuecomment-93951461 I would like to add a second picture that shows how the data is streamed between the operators for the parallelism=1 sink. We could add a all reduce as a variant to th

[GitHub] flink pull request: [FLINK-1875] Add figure explaining slots and p...

2015-04-16 Thread rmetzger
GitHub user rmetzger opened a pull request: https://github.com/apache/flink/pull/604 [FLINK-1875] Add figure explaining slots and parallelism You can merge this pull request into a Git repository by running: $ git pull https://github.com/rmetzger/flink flink1875 Alternatively