Fix fields named 'read'

[kazimuth]

Description of Changes

Fixes https://github.com/orgs/clockworklabs/projects/22?pane=issue&itemId=102392974&issue=clockworklabs%7Ccom.clockworklabs.spacetimedbsdk%7C276

by renaming internal static serializer fields so that they do not overlap with user-provided names.

Note, however, that some field names still will not work: ReadFields, WriteFields, Equals, and GetHashCode.
This would require a separate fix since the error would happen in a different place. In this case we would need to change the name of the generated member to something like ReadFields_ or Equals_, which I'm not sure is a good idea.

API and ABI breaking changes

N/A

Expected complexity level and risk

0

Testing

SDK tests and dotnet-verify tests are passing.

[joshua-spacetime]

Note, however, that some field names still will not work: ReadFields, WriteFields, Equals, and GetHashCode.
This would require a separate fix since the error would happen in a different place. In this case we would need to change the name of the generated member to something like ReadFields_ or Equals_, which I'm not sure is a good idea.

Makes sense. What do you think @jdetter?

[jdetter]

I don't super care about Equals or HashCode because I don't think we want users overriding these anyway. I think we should fix the cases for ReadFields and WriteFields because I could easily see someone creating a table that has a column with those names. (e.g. if you made a permissions table or something)

@kazimuth @joshua-spacetime does that sound good?

[jdetter]

@kazimuth Does this actually not have any API/ABI breaking changes? This doesn't require a C# SDK change?

[CLAassistant]

All committers have signed the CLA.