Paths

Table of Contentst

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

	F3344205: D13273.diff
	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

	Unknown Object (File)
	Oct 22 2024, 5:33 AM

	Unknown Object (File)
	Oct 21 2024, 8:54 AM

View All 51 Files

Subscribers

Details

Reviewers

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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

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

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

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

Path

Size

keyserver/

src/

6 lines

Diff 44060

keyserver/src/keyserver.js

Loading...