Github user uce commented on the issue:
https://github.com/apache/flink/pull/6031
> I planned to upgrade netty and drop netty-router in one step of upgrading
flink-shaded-netty. Do you think it should be split somehow?
No, if the ticket for upgrading covers that, we are all g
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/6031#discussion_r190274800
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/router/MethodlessRouter.java
---
@@ -0,0 +1,121 @@
+/*
+ * Licensed
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/6031#discussion_r190274982
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/router/RouterHandler.java
---
@@ -0,0 +1,109 @@
+/*
+ * Licensed
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/6031#discussion_r190214988
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/router/RouterHandler.java
---
@@ -0,0 +1,109 @@
+/*
+ * Licensed
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/6031#discussion_r190248429
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/router/RouteResult.java
---
@@ -0,0 +1,136 @@
+/*
+ * Licensed
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/6031#discussion_r190244366
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/router/MethodlessRouter.java
---
@@ -0,0 +1,121 @@
+/*
+ * Licensed
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/6031#discussion_r190191066
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/router/RouterHandler.java
---
@@ -0,0 +1,109 @@
+/*
+ * Licensed
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/6031#discussion_r190246117
--- Diff:
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/RuntimeMonitorHandler.java
---
@@ -89,19 +91,20 @@ public RuntimeMonitorHandler
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/6031#discussion_r190248637
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/router/Router.java
---
@@ -0,0 +1,398 @@
+/*
+ * Licensed to the Apache
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/6031#discussion_r190213387
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/router/MethodlessRouter.java
---
@@ -0,0 +1,121 @@
+/*
+ * Licensed
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/6031#discussion_r190248244
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/router/PathPattern.java
---
@@ -0,0 +1,179 @@
+/*
+ * Licensed
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/6031#discussion_r190186599
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/router/RouterHandler.java
---
@@ -0,0 +1,109 @@
+/*
+ * Licensed
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/6031#discussion_r190197601
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/rest/handler/router/MethodlessRouter.java
---
@@ -0,0 +1,121 @@
+/*
+ * Licensed
Github user uce commented on the pull request:
https://github.com/apache/flink/commit/3a61ea47922280e15f462ca3cdc0c367047bde24#commitcomment-28293221
@twalthr I think this broke the build. At least locally, I get a RAT
license failure.
```
1 Unknown Licenses
Github user uce commented on the issue:
https://github.com/apache/flink/pull/5395
Didn't merge yet, because there is an issue with the buildbot environment.
---
Github user uce commented on the issue:
https://github.com/apache/flink/pull/5395
I've built this locally and everything looks good to me (linebreaks and
code highlighting).
I will merge this to `master`, but keep this PR open for a while. If
everything works
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/4809#discussion_r166008336
--- Diff: docs/ops/state/savepoints.md ---
@@ -120,6 +120,10 @@ This will atomically trigger a savepoint for the job
with ID `:jobid` and cancel
Github user uce commented on the issue:
https://github.com/apache/flink/pull/5395
https://issues.apache.org/jira/browse/INFRA-15959
`ruby2.3.1` has been installed on the buildbot. I will check whether the
build works as expected and report back here (probably only by next
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/5395#discussion_r165295535
--- Diff: docs/_config.yml ---
@@ -77,12 +77,7 @@ defaults:
layout: plain
nav-pos: 9 # Move to end if no pos specified
Github user uce commented on the issue:
https://github.com/apache/flink/pull/5395
https://ci.apache.org/builders/flink-docs-master/builds/977/steps/Flink%20docs/logs/stdio
says
```
Ruby version:
ruby 2.0.0p384 (2014-01-12) [x86_64-linux-gnu]
```
I can ask
Github user uce commented on the issue:
https://github.com/apache/flink/pull/5331
I just tried it, re-uploading after deleting the directory **does not
work**. Good catch Stephan. :-)
@zentol: I found `HttpRequestHandler` which handles the uploads. The
handler assumes
Github user uce commented on the issue:
https://github.com/apache/flink/pull/5331
ð to merge.
---
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/5331#discussion_r162935682
--- Diff:
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/handlers/JarListHandler.java
---
@@ -77,6 +82,11 @@ public boolean accept(File
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/5331#discussion_r162932050
--- Diff:
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/handlers/JarListHandler.java
---
@@ -145,6 +155,7 @@ public boolean accept(File
Github user uce commented on the issue:
https://github.com/apache/flink/pull/5264
Build failure is unrelated (YARN test). Merging this. Thanks!
---
Github user uce commented on the issue:
https://github.com/apache/flink/pull/5264
Looks good to me. I've tried this out locally for both a working and not
working (see screenshot) JAR.
Backport of #4162 for the `release-1.2` branch.
You can merge this pull request into a Git repository by running
GitHub user uce opened a pull request:
https://github.com/apache/flink/pull/4163
[FLINK-6952] [FLINK-6985] [docs] Add Javadocs links and remove bugfix
version (1.3)
Backport of #4162 for the `release-1.3` branch.
You can merge this pull request into a Git repository by running
GitHub user uce opened a pull request:
https://github.com/apache/flink/pull/4162
[FLINK-6952] [FLINK-6985] [docs] Add Javadocs links and remove bugfix
version
Adds a link to the Javadocs in the left side navigation:
 It was interesting to follow your discussions @iemejia,
@jgrier and @patricklucas. I've tried this out and it works as expected. I'm
going to merge this later today
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/3460#discussion_r105929511
--- Diff:
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/history/HistoryServerArchiveFetcher.java
---
@@ -0,0 +1,244
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/3460#discussion_r105929296
--- Diff:
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/history/HistoryServerArchiveFetcher.java
---
@@ -0,0 +1,244
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/3460#discussion_r105928423
--- Diff:
flink-core/src/main/java/org/apache/flink/configuration/HistoryServerOptions.java
---
@@ -0,0 +1,82 @@
+/*
+ * Licensed to the Apache
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/3460#discussion_r105906846
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/jobmanager/JobManagerOptions.java
---
@@ -32,6 +32,13 @@
public static final
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/3460#discussion_r105918471
--- Diff: flink-runtime-web/web-dashboard/app/index_hs.jade ---
@@ -0,0 +1,60 @@
+//
+ Licensed to the Apache Software Foundation (ASF) under one
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/3460#discussion_r105898166
--- Diff:
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/history/HistoryServerArchiveFetcher.java
---
@@ -0,0 +1,244
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/3460#discussion_r105894483
--- Diff:
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/history/HistoryServerArchiveFetcher.java
---
@@ -0,0 +1,244
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/3460#discussion_r105876698
--- Diff:
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/history/HistoryServer.java
---
@@ -0,0 +1,276 @@
+/*
+ * Licensed
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/3460#discussion_r105894735
--- Diff:
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/history/HistoryServerArchiveFetcher.java
---
@@ -0,0 +1,244
Github user uce commented on a diff in the pull request:
https://github.com/apache/flink/pull/3460#discussion_r105877694
--- Diff: flink-runtime-web/web-dashboard/app/index_hs.jade ---
@@ -0,0 +1,60 @@
+//
+ Licensed to the Apache Software Foundation (ASF) under one
1 - 100 of 1497 matches
Mail list logo
