[GitHub] flink pull request: FLINK-3878: File cache doesn't support multipl...

2016-05-05 Thread kkrugler
GitHub user kkrugler opened a pull request: https://github.com/apache/flink/pull/1965 FLINK-3878: File cache doesn't support multiple duplicate temp direct… You can merge this pull request into a Git repository by running: $ git pull https://github.com/kkrugler/flink FLINK

[GitHub] flink issue #6101: [FLINK-9458] Unable to recover from job failure on YARN w...

2018-05-31 Thread kkrugler
Github user kkrugler commented on the issue: https://github.com/apache/flink/pull/6101 One other note - I ran into this problem, but it wasn't on YARN. It was running locally (via unit test triggered in Eclipse). ---

[GitHub] flink issue #6101: [FLINK-9458] Unable to recover from job failure on YARN w...

2018-05-31 Thread kkrugler
Github user kkrugler commented on the issue: https://github.com/apache/flink/pull/6101 I'm wondering why, now, we're encountering cases where the `sharedSlot` value is null? Seems like this could be caused by a deeper problem somewhere, so just adding the null check is masking

[GitHub] flink pull request #5848: Minor cleanup of Java example code for AsyncFuncti...

2018-04-14 Thread kkrugler
GitHub user kkrugler opened a pull request: https://github.com/apache/flink/pull/5848 Minor cleanup of Java example code for AsyncFunctions Use `CompletableFuture` correctly, via `.supplyAsync()` wrapping `Future.get()` You can merge this pull request into a Git repository

[GitHub] flink issue #5848: Minor cleanup of Java example code for AsyncFunctions

2018-04-16 Thread kkrugler
Github user kkrugler commented on the issue: https://github.com/apache/flink/pull/5848 BTW, how do people check .md editing results before pushing? I've tried a few different plugins and command line utilities, but the mix of HTML and markdown has meant none of them render properly. ---

[GitHub] flink issue #5848: Minor cleanup of Java example code for AsyncFunctions

2018-04-16 Thread kkrugler
Github user kkrugler commented on the issue: https://github.com/apache/flink/pull/5848 Hi @tedyu - not able to assign you as a reviewer, I guess I would need more Git fu to be able to do that. ---

[GitHub] flink pull request #5630: [FLINK-8849][Documentation] Fix link to chaining d...

2018-03-04 Thread kkrugler
GitHub user kkrugler opened a pull request: https://github.com/apache/flink/pull/5630 [FLINK-8849][Documentation] Fix link to chaining docs OK, after reading through the (very helpful) contributing text, it looks like you'd rather not have Jira issues for minor doc fixes, right