Skip to content

fix: make store-based barrier reusable across updates - #107

Open
RunningLeon wants to merge 1 commit into
MoonshotAI:mainfrom
RunningLeon:fix/reusable-store-barrier
Open

fix: make store-based barrier reusable across updates#107
RunningLeon wants to merge 1 commit into
MoonshotAI:mainfrom
RunningLeon:fix/reusable-store-barrier

Conversation

@RunningLeon

Copy link
Copy Markdown

fix: make the shared-store barrier reusable across updates in #106

Summary

  • add an independent generation counter for ParameterServer.store_based_barrier();
  • use a unique _store_based_barrier() group name for every invocation;
  • keep the long-lived root TCPStore unchanged and shared;
  • add CPU-only regression coverage for two consecutive barriers on the same TCPStore.

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.

1 participant