Skip to content

fix: avoid import_batch shared-dep panic#941

Open
lodystage[bot] wants to merge 2 commits intomainfrom
fix/import-batch-shared-dep-guard
Open

fix: avoid import_batch shared-dep panic#941
lodystage[bot] wants to merge 2 commits intomainfrom
fix/import-batch-shared-dep-guard

Conversation

@lodystage
Copy link
Copy Markdown

@lodystage lodystage bot commented Mar 31, 2026

Summary

  • skip duplicate VV evaluation when shared deps queue the same DAG node twice during ensure_vv_for
  • add an import_batch regression test built from public import_json_updates input so causally valid redundant deps no longer panic

Testing

  • cargo test -p loro-internal --test import_batch_shared_dep
  • cargo test -p loro-internal import_batch

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