Skip to content

Zeppelin-113 keep websocket connections alive - #109

Closed
guptarajat wants to merge 2 commits into
apache:masterfrom
guptarajat:master
Closed

Zeppelin-113 keep websocket connections alive#109
guptarajat wants to merge 2 commits into
apache:masterfrom
guptarajat:master

Conversation

@guptarajat

Copy link
Copy Markdown

Browser keeps pinging server every 60 secs to keep the websocket connection alive.

@Leemoonsoo

Copy link
Copy Markdown
Member

Thanks for contribution. LGTM!

@corneadougcorneadoug mentioned this pull request Jul 3, 2015
asfgit pushed a commit that referenced this pull request Jul 3, 2015
Ping feature to keep websocket was introduced in #109 .
But it was deleted during the merge of #56.
This is bringing it back
Author: Damien Corneau <corneadoug@gmail.com>
Closes#137 from corneadoug/fix/ping and squashes the following commits:
9677c30 [Damien Corneau] Remove spaces
4ccb246 [Damien Corneau] Fix ZEPPELIN-113
Leemoonsoo pushed a commit to Leemoonsoo/zeppelin that referenced this pull request Sep 17, 2015
Browser keeps pinging server every 60 secs to keep the websocket connection alive.
Author: Rajat Gupta <rgupta@qubole.com>
Closesapache#109 from guptarajat/master and squashes the following commits:
7627598 [Rajat Gupta] Merge remote branch 'orig/master'
987a967 [Rajat Gupta] ping pong to keep web socket alive
(cherry picked from commit 21d43c2)
Signed-off-by: Lee moon soo <moon@apache.org>
Leemoonsoo pushed a commit to Leemoonsoo/zeppelin that referenced this pull request Sep 17, 2015
Ping feature to keep websocket was introduced in apache#109 .
But it was deleted during the merge of apache#56.
This is bringing it back
Author: Damien Corneau <corneadoug@gmail.com>
Closesapache#137 from corneadoug/fix/ping and squashes the following commits:
9677c30 [Damien Corneau] Remove spaces
4ccb246 [Damien Corneau] Fix ZEPPELIN-113
(cherry picked from commit 94c7375)
Signed-off-by: Lee moon soo <moon@apache.org>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@guptarajat@Leemoonsoo