[GitHub] flink pull request: [FLINK-3711][docs] Documentation of Scala fold...

2016-04-06 Thread rehevkor5
GitHub user rehevkor5 opened a pull request: https://github.com/apache/flink/pull/1860 [FLINK-3711][docs] Documentation of Scala fold()() uses correct syntax Scala's KeyedStream#fold which accepts scala.Function2 is defined as a partially appliable function. The documentation

[GitHub] flink pull request #2341: [FLINK-4334] [quickstarts] Correctly exclude hadoo...

2016-08-08 Thread rehevkor5
GitHub user rehevkor5 opened a pull request: https://github.com/apache/flink/pull/2341 [FLINK-4334] [quickstarts] Correctly exclude hadoop1 in quickstart POMs Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list

[GitHub] flink pull request #2341: [FLINK-4334] [quickstarts] Correctly exclude hadoo...

2016-08-09 Thread rehevkor5
Github user rehevkor5 closed the pull request at: https://github.com/apache/flink/pull/2341 --- 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

[GitHub] flink issue #2341: [FLINK-4334] [quickstarts] Correctly exclude hadoop1 in q...

2016-08-09 Thread rehevkor5
Github user rehevkor5 commented on the issue: https://github.com/apache/flink/pull/2341 You got it @StephanEwen --- 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

[GitHub] flink issue #2051: [FLINK-3779] Add support for queryable state

2016-08-02 Thread rehevkor5
Github user rehevkor5 commented on the issue: https://github.com/apache/flink/pull/2051 Hi, it's great to see that someone is working on this stuff! I just wanted to put in my two cents, to provide a different perspective that might change how you are thinking about

[GitHub] flink pull request #2383: [FLINK-4418] [client] Improve resilience when Inet...

2016-08-17 Thread rehevkor5
GitHub user rehevkor5 opened a pull request: https://github.com/apache/flink/pull/2383 [FLINK-4418] [client] Improve resilience when InetAddress.getLocalHos… Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list

[GitHub] flink pull request #3189: [FLINK-5608] [webfrontend] Cancel button stays vis...

2017-01-22 Thread rehevkor5
GitHub user rehevkor5 opened a pull request: https://github.com/apache/flink/pull/3189 [FLINK-5608] [webfrontend] Cancel button stays visible in narrow windows - Most importantly, the Cancel button has been changed to float right, and will only wrap downward if pushed out

[GitHub] flink issue #3189: [FLINK-5608] [webfrontend] Cancel button stays visible in...

2017-01-25 Thread rehevkor5
Github user rehevkor5 commented on the issue: https://github.com/apache/flink/pull/3189 Thanks @uce that's helpful! Didn't think of putting screenshots in the PR. @zentol The global job state counts didn't seem like the most important thing to show when the browser is narrow

[GitHub] flink issue #3189: [FLINK-5608] [webfrontend] Cancel button stays visible in...

2017-01-27 Thread rehevkor5
Github user rehevkor5 commented on the issue: https://github.com/apache/flink/pull/3189 @zentol I updated the commit: the overall job status count is now displayed in medium-width (but not small or below), and I move the job status count to the right of job start/end & dura

[GitHub] flink issue #2383: [FLINK-4418] [client] Improve resilience when InetAddress...

2016-08-24 Thread rehevkor5
Github user rehevkor5 commented on the issue: https://github.com/apache/flink/pull/2383 @StephanEwen do you want me to take a look & see what straightforward refactoring I can make, and squeeze that into this PR? I can't tell if that was a hint or not :) --- If your project is

[GitHub] flink issue #2383: [FLINK-4418] [client] Improve resilience when InetAddress...

2016-08-18 Thread rehevkor5
Github user rehevkor5 commented on the issue: https://github.com/apache/flink/pull/2383 I don't think the Jenkins build failure was due to my changes. --- 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

[GitHub] flink issue #3059: [docs] Clarify restart strategy defaults set by checkpoin...

2017-01-06 Thread rehevkor5
Github user rehevkor5 commented on the issue: https://github.com/apache/flink/pull/3059 The changes so far are docs-only, so the CI failure must be unrelated. --- 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

[GitHub] flink pull request #3079: [FLINK-5424] Improve Restart Strategy Logging

2017-01-06 Thread rehevkor5
GitHub user rehevkor5 opened a pull request: https://github.com/apache/flink/pull/3079 [FLINK-5424] Improve Restart Strategy Logging - Added toString for FailureRateRestartStrategy (important for JobManager's "Using restart strategy $restartStrategy" log message)

[GitHub] flink issue #3059: [docs] Clarify restart strategy defaults set by checkpoin...

2017-01-09 Thread rehevkor5
Github user rehevkor5 commented on the issue: https://github.com/apache/flink/pull/3059 @uce Thanks. I didn't get an opportunity to address your comments... is there anything you want me to do? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] flink issue #3059: [docs] Clarify restart strategy defaults set by checkpoin...

2017-01-09 Thread rehevkor5
Github user rehevkor5 commented on the issue: https://github.com/apache/flink/pull/3059 I'm curious as to how you added my commit to `master` and `release-1.2` but it doesn't show up in this PR. The whole "rehevkor5 committed with uce 5 days ago" thing. Github i

[GitHub] flink issue #3059: [docs] Clarify restart strategy defaults set by checkpoin...

2017-01-09 Thread rehevkor5
Github user rehevkor5 commented on the issue: https://github.com/apache/flink/pull/3059 It does, thanks! I'll try to remember to include "closes #_{pr number}_" in my commit messages which might be handy. --- If your project is set up for it, you can reply to this emai

[GitHub] flink pull request #3059: [docs] Clarify restart strategy defaults set by ch...

2017-01-04 Thread rehevkor5
GitHub user rehevkor5 opened a pull request: https://github.com/apache/flink/pull/3059 [docs] Clarify restart strategy defaults set by checkpointing - Added info about checkpointing changing the default restart strategy in places where it was missing: the config page