[PR #673] [MERGED] Version all channel changes #1598

Closed
opened 2026-02-20 17:45:07 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langchain-ai/langgraph/pull/673
Author: @nfcampos
Created: 6/14/2024
Status: Merged
Merged: 6/14/2024
Merged by: @nfcampos

Base: mainHead: nc/14jun/version-all-channel-changes


📝 Commits (2)

📊 Changes

14 files changed (+132 additions, -70 deletions)

View changed files

📝 langgraph/channels/any_value.py (+4 -3)
📝 langgraph/channels/base.py (+6 -5)
📝 langgraph/channels/binop.py (+3 -3)
📝 langgraph/channels/context.py (+2 -1)
📝 langgraph/channels/dynamic_barrier_value.py (+10 -3)
📝 langgraph/channels/ephemeral_value.py (+4 -4)
📝 langgraph/channels/last_value.py (+3 -2)
📝 langgraph/channels/named_barrier_value.py (+9 -3)
📝 langgraph/channels/topic.py (+7 -1)
📝 langgraph/checkpoint/sqlite.py (+6 -2)
📝 langgraph/pregel/__init__.py (+31 -5)
📝 tests/test_channels.py (+43 -34)
📝 tests/test_pregel.py (+2 -2)
📝 tests/test_pregel_async.py (+2 -2)

📄 Description

  • previously when channels were cleared via update([]) or consume() that would not bump the version
  • that behavior is not compatible with checkpointers that store blobs separately, which join checkpoint and blobs using version
  • now update() and consume() return bool to indicate if version should be bumped
  • this also avoids bumping version when nothing changed

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langchain-ai/langgraph/pull/673 **Author:** [@nfcampos](https://github.com/nfcampos) **Created:** 6/14/2024 **Status:** ✅ Merged **Merged:** 6/14/2024 **Merged by:** [@nfcampos](https://github.com/nfcampos) **Base:** `main` ← **Head:** `nc/14jun/version-all-channel-changes` --- ### 📝 Commits (2) - [`758d62f`](https://github.com/langchain-ai/langgraph/commit/758d62fd22ac14e8ebd208eeec80ea9ad378d903) Version all channel changes - [`4ff56ed`](https://github.com/langchain-ai/langgraph/commit/4ff56edc26adf3cb45fdeee03bde579178baae24) Update docstrings ### 📊 Changes **14 files changed** (+132 additions, -70 deletions) <details> <summary>View changed files</summary> 📝 `langgraph/channels/any_value.py` (+4 -3) 📝 `langgraph/channels/base.py` (+6 -5) 📝 `langgraph/channels/binop.py` (+3 -3) 📝 `langgraph/channels/context.py` (+2 -1) 📝 `langgraph/channels/dynamic_barrier_value.py` (+10 -3) 📝 `langgraph/channels/ephemeral_value.py` (+4 -4) 📝 `langgraph/channels/last_value.py` (+3 -2) 📝 `langgraph/channels/named_barrier_value.py` (+9 -3) 📝 `langgraph/channels/topic.py` (+7 -1) 📝 `langgraph/checkpoint/sqlite.py` (+6 -2) 📝 `langgraph/pregel/__init__.py` (+31 -5) 📝 `tests/test_channels.py` (+43 -34) 📝 `tests/test_pregel.py` (+2 -2) 📝 `tests/test_pregel_async.py` (+2 -2) </details> ### 📄 Description - previously when channels were cleared via update([]) or consume() that would not bump the version - that behavior is not compatible with checkpointers that store blobs separately, which join checkpoint and blobs using version - now update() and consume() return bool to indicate if version should be bumped - this also avoids bumping version when nothing changed --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-20 17:45:07 -05:00
yindo closed this issue 2026-02-20 17:45:07 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langchain-ai/langgraph#1598