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

Adding postgres cache support for node #1648

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

zhouyuf6741
Copy link
Collaborator

Summary:
We currently have writes result stored to the mapping data storage. however, during each different execution, we still rerun the execution to get the writes result.
To solve this issue, we provided a CachePolicy which allows people pass in a cache_key which specify how they want to cache this node. It can based up the step name or user id etc.

This version currently only support postgres sync request, but in the future we would like to expand this to support all the data storages and async.

Test:
Added test_cache file.

@zhouyuf6741 zhouyuf6741 force-pushed the cache-key-node branch 5 times, most recently from c1cd90f to 23e85e8 Compare September 7, 2024 02:15
@zhouyuf6741
Copy link
Collaborator Author

zhouyuf6741 commented Sep 7, 2024

@nfcampos looks like test_pregel.py::test_message_graph is failing for langchain-core: 0.3.0.dev1 even on the main branch. Let me know if you want me to update the snapshot.

@vbarda
Copy link
Collaborator

vbarda commented Sep 8, 2024

@nfcampos looks like test_pregel.py::test_message_graph is failing for langchain-core: 0.3.0.dev1 even on the main branch. Let me know if you want me to update the snapshot.

this should be fixed now -- merged the PR fixing the snapshots

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.

2 participants