Skip to content
This repository has been archived by the owner on Apr 14, 2023. It is now read-only.

Use ws also for regular queries #68

Closed
wmertens opened this issue Feb 6, 2017 · 2 comments
Closed

Use ws also for regular queries #68

wmertens opened this issue Feb 6, 2017 · 2 comments

Comments

@wmertens
Copy link

wmertens commented Feb 6, 2017

Since you are setting up the websocket anyway, you can use that channel to request normal queries as well, leading to less overhead (no TCP setup, no session lookup).
@DxCx
Copy link
Contributor

DxCx commented Feb 6, 2017

hi @wmertens please see apollographql/apollo-server#272 :)

@wmertens
Copy link
Author

wmertens commented Feb 6, 2017

Well I'll close this one then 😄

@wmertens wmertens closed this as completed Feb 6, 2017
martijnwalraven pushed a commit that referenced this issue Oct 17, 2017
Merge pull request #68 from matthiasklan/patch-1
9a8597b 
Update subscriptions-to-schema.md
baconz pushed a commit to PhiloInc/subscriptions-transport-ws that referenced this issue Apr 24, 2018
@helfer
Merge pull request apollographql#68 from apollostack/more-tests
5b1575d 
More tests to increase test coverage
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@wmertens @DxCx