SpacetimeDB icon indicating copy to clipboard operation
SpacetimeDB copied to clipboard

Revisit reserved identifiers

Open kazimuth opened this issue 1 year ago • 5 comments

Currently I've removed reserved identifiers, this should be reworked somehow before 1.0

kazimuth avatar Sep 19 '24 18:09 kazimuth

Adding this to 1.0 release

bfops avatar Jan 13 '25 18:01 bfops

This is a breaking-ish change (because users may have used a word that we might end up deciding to reserve).

bfops avatar Jan 13 '25 20:01 bfops

See https://github.com/clockworklabs/SpacetimeDBPrivate/blob/master/proposals/0026-module-and-sdk-api/02-physical-queries.md#reserved-column-and-index-accessor-names and https://github.com/clockworklabs/SpacetimeDBPrivate/blob/master/proposals/0026-module-and-sdk-api/05-user-defined-reducers.md#reserved-reducer-names . We have no plans to reserve any other words at this time. Notably we do not want to reserve the words reserved by the SQL spec.

gefjon avatar Jan 13 '25 20:01 gefjon

Let's also enforce this in the rust macro system. (It's already done in C#)

bfops avatar Jan 27 '25 21:01 bfops

We think we can get this in as a "bugfix" down the line, even though it will technically be a breaking change. We definitely can't get it into 1.0.

gefjon avatar Feb 07 '25 18:02 gefjon