[ 
https://issues.apache.org/jira/browse/FLINK-7738?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16191413#comment-16191413
 ] 

ASF GitHub Bot commented on FLINK-7738:
---------------------------------------

Github user EronWright commented on a diff in the pull request:

    https://github.com/apache/flink/pull/4767#discussion_r142700416
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/rest/RestClient.java ---
    @@ -275,4 +301,121 @@ public HttpResponseStatus getHttpResponseStatus() {
                        return httpResponseStatus;
                }
        }
    +
    +   public <M extends MessageHeaders<EmptyRequestBody, 
WebSocketUpgradeResponseBody, U>, U extends MessageParameters, R extends 
ResponseBody> CompletableFuture<WebSocket> sendWebSocketRequest(String 
targetAddress, int targetPort, M messageHeaders, U messageParameters, Class<R> 
messageClazz, WebSocketListener... listeners) throws IOException {
    --- End diff --
    
    I too was unhappy about using a special response body, but felt that the 
alternative required some rework of the REST handler that was best done in a 
follow-up.  With some rework we can eliminate the funky response body.


> Create WebSocket handler (server)
> ---------------------------------
>
>                 Key: FLINK-7738
>                 URL: https://issues.apache.org/jira/browse/FLINK-7738
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Cluster Management, Mesos
>            Reporter: Eron Wright 
>            Assignee: Eron Wright 
>
> An abstract handler is needed to support websocket communication.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to