[PR #341] [MERGED] loosen restrictions on chain version in batch read scenarios #378

Closed
opened 2026-10-26 09:49:26 +00:00 by bsakaguchi · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/deepseek-ai/3FS/pull/341
Author: @SF-Zhou
Created: 9/23/2025
Status: ✅ Merged
Merged: 9/23/2025
Merged by: @SF-Zhou

Base: main ← Head: SF-Zhou-patch-1


📝 Commits (1)

  • 4998f56 loosen restrictions on chain version in batch read scenarios

📊 Changes

1 file changed (+1 additions, -1 deletions)

View changed files

📝 src/storage/service/TargetMap.cc (+1 -1)

📄 Description

In the context of batch read, if a client can initiate a read request to a server and the target status on the server side is valid, it means that under the current routing versions of the client and server, the read request is considered valid. Therefore, the additional chain version check has been removed.

Fix #340.


🔄 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/deepseek-ai/3FS/pull/341 **Author:** [@SF-Zhou](https://github.com/SF-Zhou) **Created:** 9/23/2025 **Status:** ✅ Merged **Merged:** 9/23/2025 **Merged by:** [@SF-Zhou](https://github.com/SF-Zhou) **Base:** `main` ← **Head:** `SF-Zhou-patch-1` --- ### 📝 Commits (1) - [`4998f56`](https://github.com/deepseek-ai/3FS/commit/4998f56905cee6c60bb4e350b3831e4fb1460de2) loosen restrictions on chain version in batch read scenarios ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `src/storage/service/TargetMap.cc` (+1 -1) </details> ### 📄 Description In the context of batch read, if a client can initiate a read request to a server and the target status on the server side is valid, it means that under the current routing versions of the client and server, the read request is considered valid. Therefore, the additional chain version check has been removed. Fix #340. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
bsakaguchi 2026-10-26 09:49:26 +00:00
Sign in to join this conversation.
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
deepseek-ai/3FS#378
No description provided.