Page MenuHomePhabricator
Differential D13273

[keyserver] convert secondary migration block from do while to regular while loop
ClosedPublic
Actions

Authored by will on Sep 9 2024, 12:47 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 22, 3:50 PM
Unknown Object (File)
Fri, Nov 22, 3:50 PM
Unknown Object (File)
Fri, Nov 22, 4:31 AM
Unknown Object (File)
Sun, Nov 10, 4:49 AM
Unknown Object (File)
Sat, Nov 9, 6:28 PM
Unknown Object (File)
Sat, Nov 9, 8:11 AM
Unknown Object (File)
Oct 22 2024, 6:29 PM
Unknown Object (File)
Oct 22 2024, 5:34 AM
View All 53 Files
Subscribers
tomek

Details

Reviewers
ashoat
Commits
rCOMM7488a4e6fc3a: [keyserver] convert secondary migration block from do while to regular while…
Summary

The do-while ensures an unnecessary 5 second sleep time. Converting this to a while loop ensures this only happens if necessary

Test Plan

flow check

Diff Detail

Repository
rCOMM Comm
Branch
flip_sleep_version_check
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

will created this revision.Sep 9 2024, 12:47 PM
Herald added a subscriber: tomek. · View Herald TranscriptSep 9 2024, 12:47 PM
will mentioned this in D13213: [keyserver] stall secondary nodes until database version is equal to or greater than latest wrapped in transaction request blocking migration version..Sep 9 2024, 12:48 PM
Harbormaster completed remote builds in B31506: Diff 44003.Sep 9 2024, 1:04 PM
will requested review of this revision.Sep 9 2024, 1:04 PM
ashoat requested changes to this revision.Sep 9 2024, 1:38 PM

Good point here – can we avoid the sleep unless we need it? I think my feedback here was misguided... we should probably just revert to the version of the code you had before that feedback

This revision now requires changes to proceed.Sep 9 2024, 1:38 PM
will updated this revision to Diff 44056.Sep 11 2024, 12:47 PM

review feedback

will retitled this revision from [keyserver] flip sleep and db version check to [keyserver] convert secondary migration block from do while to regular while loop.Sep 11 2024, 12:48 PM
will edited the summary of this revision. (Show Details)
Harbormaster completed remote builds in B31547: Diff 44056.Sep 11 2024, 1:04 PM
ashoat accepted this revision.Sep 11 2024, 1:07 PM

Thanks, and sorry for my confusion!

This revision is now accepted and ready to land.Sep 11 2024, 1:07 PM
Closed by commit rCOMM7488a4e6fc3a: [keyserver] convert secondary migration block from do while to regular while… (authored by will). · Explain WhySep 11 2024, 1:56 PM
This revision was automatically updated to reflect the committed changes.
will added a commit: rCOMM7488a4e6fc3a: [keyserver] convert secondary migration block from do while to regular while….

Revision Contents
Changeset List

PathSize
keyserver/
src/
6 lines

Diff 44056

View Options

keyserver/src/keyserver.js

Loading...