Page MenuHomePhabricator
Differential D10215

[identity] Implemet attribute conversions for DeviceList row
ClosedPublic
Actions

Authored by bartek on Dec 7 2023, 12:05 AM.
Tags
None
Referenced Files
F6238581: D10215.id34365.diff
Thu, Apr 24, 6:27 PM
F6238578: D10215.id.diff
Thu, Apr 24, 6:27 PM
Unknown Object (File)
Tue, Apr 22, 1:13 PM
Unknown Object (File)
Sun, Apr 20, 7:45 AM
Unknown Object (File)
Sat, Apr 19, 11:28 PM
Unknown Object (File)
Sat, Apr 19, 10:28 PM
Unknown Object (File)
Sat, Apr 19, 7:58 PM
Unknown Object (File)
Sat, Apr 19, 2:32 AM
View All 95 Files
Subscribers
ashoat
tomek

Details

Reviewers
michal
varun
Commits
rCOMMbffebde31833: [identity] Implemet attribute conversions for DeviceList row
Summary

Same as D10214 but for DeviceListRow

Depends on D10214

Test Plan

Tested later in the stack. TODO: add unit tests

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bartek created this revision.Dec 7 2023, 12:05 AM
bartek held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptDec 7 2023, 12:05 AM
bartek added a child revision: D10216: identity] Add function to get current device list timestamp.Dec 7 2023, 12:09 AM
Harbormaster failed remote builds in B24843: Diff 34365!Dec 7 2023, 1:02 AM
bartek published this revision for review.Dec 7 2023, 1:05 AM
bartek added inline comments.
services/identity/src/database/device_list.rs
280–287

Note that we use RFC 3339 string for ItemID while the ATTR_TIMESTAMP uses milliseconds number. Generally I use the former everywhere (simpler parsing without NaiveDateTime conversions) but I thought the latter is easier for DDB to sort by (it's used only as index sorting key and not read anywhere else)

varun accepted this revision.Dec 7 2023, 9:23 PM
This revision is now accepted and ready to land.Dec 7 2023, 9:23 PM
bartek updated this revision to Diff 34500.Dec 11 2023, 11:27 PM

Rebase

Harbormaster completed remote builds in B24962: Diff 34500.Dec 12 2023, 12:09 AM
Closed by commit rCOMMbffebde31833: [identity] Implemet attribute conversions for DeviceList row (authored by bartek). · Explain WhyJan 5 2024, 1:22 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMMbffebde31833: [identity] Implemet attribute conversions for DeviceList row.

Revision Contents
Changeset List

PathSize
services/
identity/
src/
database/
113 lines

Diff 34365

View Options

services/identity/src/database/device_list.rs

Loading...