-
-
Notifications
You must be signed in to change notification settings - Fork 323
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
Parallel cache push strawman #4467
Conversation
@bdbaddog - This seems to pass all tests, but please let me know if you would prefer to do this under a |
One concern/question. Would restoring .cached break anything in this change? |
I don't think it would break anything to restore it. I took it out mostly because as far as I could see it was only being used to transfer info about caching from |
I think it's worth adding back in. |
This partially reverts commit 8a6b5e4 to restore `Node.cached` field.
4accdb3
to
e0272d2
Compare
@bdbaddog - done. |
Contributor Checklist:
CHANGES.txt
(and read theREADME.rst
)