Page MenuHomePhabricator
Differential D7718

[Tunnelbroker] Create undelivered messages table
ClosedPublic
Actions

Authored by • jon on May 4 2023, 9:52 AM.
Tags
None
Referenced Files
F1569324: D7718.id27019.diff
Thu, Apr 18, 3:18 AM
F1569323: D7718.id26428.diff
Thu, Apr 18, 3:18 AM
F1569322: D7718.id26080.diff
Thu, Apr 18, 3:18 AM
F1569321: D7718.id26132.diff
Thu, Apr 18, 3:18 AM
F1569295: D7718.id.diff
Thu, Apr 18, 3:18 AM
F1569281: D7718.diff
Thu, Apr 18, 3:12 AM
Unknown Object (File)
Mar 2 2024, 1:45 PM
Unknown Object (File)
Feb 11 2024, 4:40 AM
View All 38 Files
Subscribers
ashoat
atul
tomek

Details

Reviewers
varun
bartek
Commits
rCOMMfa4a6ccb98f6: [Tunnelbroker] Create undelivered messages table
Summary

Create dynamoDB table for storing undelivered
messages

Part of: https://linear.app/comm/issue/ENG-3823

Test Plan
# start localstack
comm-dev services start

# ensure that tunnelbroker-undelivered-messages table is created
cd services/terraform
./run.sh

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

• jon created this revision.May 4 2023, 9:52 AM
Herald added subscribers: atul, tomek, ashoat. · View Herald TranscriptMay 4 2023, 9:52 AM
Harbormaster returned this revision to the author for changes because remote builds failed.May 4 2023, 10:07 AM
Harbormaster failed remote builds in B19048: Diff 26080!
• jon added a parent revision: D7717: [Tunnelbroker] Remove existing tunnelbroker tables.May 5 2023, 1:36 PM
• jon added a child revision: D7720: [Tunnelbroker] Define undelivered-messages table in rust.
• jon updated this revision to Diff 26132.May 5 2023, 1:41 PM

Add createdAt as range_key

Harbormaster completed remote builds in B19084: Diff 26132.May 5 2023, 2:03 PM
• jon requested review of this revision.May 5 2023, 2:03 PM
bartek added inline comments.May 8 2023, 7:23 AM
services/terraform/dynamodb.tf
101–104 ↗(On Diff #26132)

In other places, we use the RFC3339 string as a timestamp. Not sure if it is a good candidate for a range key though

• jon marked an inline comment as done.May 9 2023, 11:17 PM
• jon added inline comments.
services/terraform/dynamodb.tf
101–104 ↗(On Diff #26132)

I just intend to use this to order the messages, so seconds from EPOCH seems like a simple yet effective way to achieve this.

This also avoids having to parse the date back in.

bartek accepted this revision.May 9 2023, 11:18 PM
bartek added inline comments.
services/terraform/dynamodb.tf
101–104 ↗(On Diff #26132)

That makes sense, thanks for explaining

This revision is now accepted and ready to land.May 9 2023, 11:18 PM
• jon marked 2 inline comments as done.May 12 2023, 4:28 PM
• jon updated this revision to Diff 26428.May 12 2023, 4:32 PM

Rebase on master

Harbormaster completed remote builds in B19314: Diff 26428.May 12 2023, 4:50 PM
Closed by commit rCOMMfa4a6ccb98f6: [Tunnelbroker] Create undelivered messages table (authored by • jon). · Explain WhyMay 24 2023, 2:36 PM
This revision was automatically updated to reflect the committed changes.
• jon added a commit: rCOMMfa4a6ccb98f6: [Tunnelbroker] Create undelivered messages table.

Revision Contents
Changeset List

PathSize
services/
terraform/
18 lines

Diff 26428

View Options

services/terraform/dynamodb.tf

Loading...