Skip to content
This repository has been archived by the owner on Apr 25, 2024. It is now read-only.

Restrict event key type #31

Merged
merged 2 commits into from
Dec 11, 2023
Merged

Restrict event key type #31

merged 2 commits into from
Dec 11, 2023

Conversation

slinkydeveloper
Copy link
Contributor

Consequence of restatedev/restate#955

slinkydeveloper added a commit to slinkydeveloper/restate that referenced this pull request Dec 8, 2023
Remove some old service definitions previously used for testing
Copy link
Contributor

@tillrohrmann tillrohrmann left a comment

Choose a reason for hiding this comment

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

LGTM. +1 for merging.

@slinkydeveloper slinkydeveloper merged commit 06ff8cc into main Dec 11, 2023
2 checks passed
@slinkydeveloper slinkydeveloper deleted the restrict_event_key_type branch December 11, 2023 08:30
slinkydeveloper added a commit to restatedev/restate that referenced this pull request Dec 12, 2023
* Forbid registration with non-string type keys
* Make sure we generate a string version of the uuid for unkeyed service key
* Remove the service_key poli views
* Remove the service key udfs
* Update protos as in restatedev/proto#31
* Propagate dev.event.Event changes.
* Remove length delimiter 
* Remove varint encoded length at the beginning of the key
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants