-
Notifications
You must be signed in to change notification settings - Fork 7
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
Consider removing autoStart, or at least disabling it by default #25
Comments
Related to this. If we invoke |
This could occur when the query has 2 or more triple patterns. Closes #777
As I mentioned in comunica/comunica#907 - it may be useful have this change made prior to the v2 release of Comunica. @rubensworks you noted that "making this change in AsyncIterator might not be trivial." - what kind of problems are you envisioning? |
Not sure. I just remember spending a lot of time debugging things related to @RubenVerborgh might also have some thoughts about this. |
I'm happy to have this change—is there still time? Can release as a v4.0.0-beta first. |
Also wondering about time. I hope to enter beta testing for Comunica 2.x later this week. |
If we release the beta of asynciterator today I can spend tommorow seeing trying to upgrade comunica - if it's too hard/time consuming then we can just leave it out? |
Sounds good to me, if you think it's feasible to flip the default value of |
No description provided.
The text was updated successfully, but these errors were encountered: