[keyserver] Create toggle pin messages and updates in parallel
ClosedPublic
Actions

Authored by rohan on Aug 4 2023, 3:53 PM.

Details

Reviewers

atul
ginsu
ashoat

Commits

rCOMM781c2d3a747a: [keyserver] Create toggle pin messages and updates in parallel

Summary

Some feedback here was to try to
create the new messages and the updates in parallel by passing two
separate async IIFEs into a Promise.all([...]).

Depends on D8732

Test Plan

Tested to make sure the new messages and the updates still
occur as before

Diff Detail

Repository

rCOMM Comm

Branch

toggle_pin_error

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

rohan created this revision.Aug 4 2023, 3:53 PM

Herald added a subscriber: tomek. · View Herald TranscriptAug 4 2023, 3:53 PM

Harbormaster completed remote builds in B21539: Diff 29547.Aug 4 2023, 4:11 PM

rohan requested review of this revision.Aug 4 2023, 4:11 PM

ashoat accepted this revision.Aug 4 2023, 4:21 PM

This revision is now accepted and ready to land.Aug 4 2023, 4:21 PM

Closed by commit rCOMM781c2d3a747a: [keyserver] Create toggle pin messages and updates in parallel. · Explain WhyAug 7 2023, 3:06 PM

This revision was automatically updated to reflect the committed changes.

rohan added a commit: rCOMM781c2d3a747a: [keyserver] Create toggle pin messages and updates in parallel.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

updaters/

thread-updaters.js

52 lines

Diff 29547

View Options

keyserver/src/updaters/thread-updaters.js

[keyserver] Create toggle pin messages and updates in parallelClosedPublicActions