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

Ability to pause and resume parsing while streaming #26

Closed
mholt opened this issue Mar 4, 2014 · 3 comments

Comments

Projects
None yet
1 participant
@mholt
Copy link
Owner

commented Mar 4, 2014

In case AJAX requests need to be made in the step function, it would be useful to stop parsing and then resume it later, when the request completes, for instance.

@mholt mholt added the enhancement label Mar 4, 2014

@mholt mholt self-assigned this Mar 4, 2014

@mholt mholt added the deferred label May 8, 2014

@mholt mholt added this to the 3.0 milestone May 8, 2014

@mholt mholt added suggestion and removed enhancement labels May 10, 2014

@mholt mholt added enhancement and removed deferred labels May 30, 2014

@mholt

This comment has been minimized.

Copy link
Owner Author

commented May 30, 2014

This has been implemented in the version 3.0 rewrite.

@mholt mholt closed this May 30, 2014

@mholt

This comment has been minimized.

Copy link
Owner Author

commented Jul 11, 2014

Well phooey. The way I restructured the code in version 3 makes this more difficult than I had anticipated. Or at least, it requires a different approach than I've attempted. Maybe I just need to pass the step function a higher-level scope than just the core Parser. I'll re-visit this, but probably not for version 3.0.

@mholt mholt reopened this Jul 11, 2014

@mholt mholt removed this from the 3.0 milestone Jul 11, 2014

@mholt mholt removed their assignment Jul 16, 2014

@mholt

This comment has been minimized.

Copy link
Owner Author

commented Sep 1, 2014

Okay, this is almost ready. Pause/resume is supported for streaming only (meaning when using the step callback) and does not work with web workers.

@mholt mholt closed this in b117772 Sep 1, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.