clockworklabs / clockworklabs/SpacetimeDB

Reimplement: websocket: send/receive reducer & table ids instead of names (take 2)

Offen
#4,792 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

api-break
Vorherrschende Sprache
Rust
Sterne
25.2k
Forks
1.1k
Ø Merge
2 T. 7 Std.
Gemergte PRs (30 T.)
46

Beschreibung

This issue tracks reimplementation of the work from stale PR #1940, which is being closed because it is too out of date to merge directly.

    Original PR: https://github.com/clockworklabs/SpacetimeDB/pull/1940
    Original author: @Centril
    Original branch: `centril/name-keyed-websocket-use-ids`
    Base branch: `master`

    ## Original PR summary

    # Description of Changes

Receive and send reducer ids and table ids as opposed to names in the SDK.

Alternative to https://github.com/clockworklabs/SpacetimeDB/pull/1883 which is more efficient, but also uses indices internally rather than strings, making the diff larger.

API and ABI breaking changes

Yes.

    ## Follow-up

    - Reimplement this change in a fresh PR against current `master`.
    - Carry forward any still-relevant context from the original PR discussion and review.
    - Link the new implementation PR back to the original stale PR for historical context.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit der Durchsicht des veralteten PR #1940, seiner Diskussion und des Branches centril/name-keyed-websocket-use-ids, und vergleiche anschließend den Vorschlag mit dem aktuellen master. Implementiere die weiterhin relevante Änderung in einem neuen PR erneut, berücksichtige den dokumentierten API- und ABI-Bruch und verlinke die neue Implementierung zurück auf PR #1940.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
rust
Bereich
api, backend-api-design
Issue-Typ
Refactoring
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.