Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: don't store duplicated message columns #581

Merged
merged 1 commit into from
Jan 29, 2024

Conversation

pjenvey
Copy link
Member

@pjenvey pjenvey commented Jan 27, 2024

  • channel_id/topic/sortkey_timestamp (in the row key)
  • expiry (in the cell timestamps)

(we can always add these back to messages later if they are worth having)

  • make add_user conditional, erroring out if the user already exists
  • utilize row_limit in message reads
  • enforce non null columns on deserialization

Closes: SYNC-4070
Closes: SYNC-4099
Closes: SYNC-4106

@pjenvey pjenvey requested a review from jrconlin January 27, 2024 00:43
@pjenvey pjenvey force-pushed the feat/bigtable-schema-cleanup2-SYNC-4070 branch from 74ba4f6 to e141505 Compare January 27, 2024 00:44
- channel_id/topic/sortkey_timestamp (in the row key)
- expiry (in the cell timestamps)

(we can always add these back to messages later if they are worth having)

- make add_user conditional, erroring out if the user already exists
- utilize row_limit in message reads
- enforce non null columns on deserialization

Closes: SYNC-4070
Closes: SYNC-4099
Closes: SYNC-4106
@pjenvey pjenvey force-pushed the feat/bigtable-schema-cleanup2-SYNC-4070 branch from e141505 to b581438 Compare January 27, 2024 23:51
@pjenvey pjenvey merged commit 7f2396a into master Jan 29, 2024
1 check passed
@pjenvey pjenvey deleted the feat/bigtable-schema-cleanup2-SYNC-4070 branch January 29, 2024 19:32
jrconlin pushed a commit that referenced this pull request Jan 29, 2024
- channel_id/topic/sortkey_timestamp (in the row key)
- expiry (in the cell timestamps)

(we can always add these back to messages later if they are worth
having)

- make add_user conditional, erroring out if the user already exists
- utilize row_limit in message reads
- enforce non null columns on deserialization

Closes: SYNC-4070
Closes: SYNC-4099
Closes: SYNC-4106
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.

2 participants