Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

transport fix for live query #43

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

imdark
Copy link

@imdark imdark commented Jul 17, 2015

This resolves #36

The connection used to only read a single chunk of data when new data is recived.
So that if a buffer was larger then a single operation is recived it will cause all updates to lag indeifinitly.
This couses issues for the live query since it often recives an update for each registeration on the same connecion, I changed the code to parse the entire buffer til its end but only one chunk at a time.

unknown and others added 4 commits July 17, 2015 21:41
is recived, so that if a buffer larger the a single operation is recived
it will cause all updates to lag, this couses issues for the live query
since it often recives an update for each registeration on the same
connecion, changed it to parse the entire buffer til its end but only one
chunk at a time
@lvca
Copy link
Member

lvca commented Jul 20, 2015

Tests on Travis won't pass. Could you please fix them to merge it?

@lvca lvca self-assigned this Jul 20, 2015
imdark added 2 commits July 22, 2015 22:49
…allow

for context switching to not take too much time on the main js thread
@imdark
Copy link
Author

imdark commented Jul 23, 2015

I have no idea why this change couses all those issues, researching it, ideas welcome

@joaobarcia
Copy link

Hi @lvca ,

Any progress on this issue? Is it still assigned to 2.2? It seems to be the only issue preventing OrientDB from being integrated into Meteor.

@imdark seems to be unresponsive this past few weeks.

Cheers

Sign up for free to 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.

3 participants