Skip to content

Batch index updates #336

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

Closed

Conversation

kazimuth
Copy link
Contributor

Description of Changes

Rearranges some loops so that each row doesn't have to make a bunch of indirect function calls to update indexes.

API

  • This is an API breaking change to the SDK

If the API is breaking, please state below what will break

Requires SpacetimeDB PRs

List any PRs here that are required for this SDK change to work

Testsuite

SpacetimeDB branch name: master

Testing

  • Walked around in BitCraft
  • Full BitCraft tutorial

@kazimuth kazimuth requested a review from rekhoff June 25, 2025 15:55
Copy link
Contributor

@rekhoff rekhoff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes make sense for the described goal, are clear, and are well commented.

@bfops
Copy link
Collaborator

bfops commented Aug 8, 2025

Migrated to clockworklabs/SpacetimeDB#3145, as we are merging this repo into that one.

@bfops bfops closed this Aug 8, 2025
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.

3 participants