Skip to content

Fix example and lots-of-data scripts - #55

Draft
ecordell wants to merge 1 commit into
mainfrom
evan/eng-876-examples
Draft

ecordell wants to merge 1 commit into
mainfrom
evan/eng-876-examples

Conversation

@ecordell

Copy link
Copy Markdown

Two bundled scripts fail against current SpiceDB.

  • lots-of-data.yaml sent 1003 updates in one write, above the SpiceDB limit of 1000. It now writes the 3 parent relationships and the 1000 tokens in two calls. The data volume does not change.
  • The checkbulk script in example.yaml expected permissions that the seeded data does not give. One check raced the write and delete scripts. The others checked reader, where the data gives view or edit. It now checks view and edit on the seeded firstdoc relationships.

Testing: both scripts ran against spicedb serve-testing with 0 errors (lots-of-data with a smaller iteration count). go test ./... and golangci-lint pass.

lots-of-data.yaml wrote 1003 updates per call, above the SpiceDB limit
of 1000. Write the three parent relationships and the 1000 tokens as
separate calls. The data volume does not change.

The checkbulk script in example.yaml expected permissions that the
data does not give: one check raced the write and delete scripts, and
the others used the reader relation where the data needs view or edit.
Check view and edit on the seeded firstdoc relationships instead.
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