Paths

Table of Contentst

Differential D7288

[web-db] extract worker write requests to a separate condition
ClosedPublic
Actions

Authored by kamil on Apr 3 2023, 5:04 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Apr 11, 8:09 PM

	Unknown Object (File)
	Wed, Apr 10, 4:46 AM

	Unknown Object (File)
	Wed, Apr 3, 8:02 AM

	Unknown Object (File)
	Wed, Apr 3, 8:02 AM

	Unknown Object (File)
	Wed, Apr 3, 8:02 AM

	Unknown Object (File)
	Wed, Apr 3, 8:02 AM

	Unknown Object (File)
	Wed, Apr 3, 8:01 AM

	Unknown Object (File)
	Wed, Apr 3, 7:55 AM

View All 78 Files

Subscribers

Details

Reviewers

michal
tomek

Commits

rCOMM49c2a6e22823: [web-db] extract worker write requests to a separate condition

Summary

Improving code by extracting common code to separate branch.

Depends on D7186

Test Plan

yarn flow check

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Apr 3 2023, 5:04 AM

kamil held this revision as a draft.

Herald added subscribers: atul, ashoat. · View Herald TranscriptApr 3 2023, 5:04 AM

kamil mentioned this in D7186: [web-db] start persisting SQLite content.Apr 3 2023, 5:05 AM

make array read only

Harbormaster completed remote builds in B17940: Diff 24572.Apr 3 2023, 5:20 AM

Harbormaster completed remote builds in B17941: Diff 24573.Apr 3 2023, 5:32 AM

kamil published this revision for review.Apr 3 2023, 5:37 AM

kamil added a child revision: D7289: [web-db] improve persisting lifecycle.Apr 3 2023, 5:46 AM

tomek accepted this revision.Apr 4 2023, 2:11 AM

tomek added inline comments.

web/database/worker/db-worker.js
207–231 ↗	(On Diff #24573)	What do you think about having early exit? We can also consider extracting write operations to a separate function that only accepts `workerWriteRequests`

This revision is now accepted and ready to land.Apr 4 2023, 2:11 AM

early exit

Harbormaster completed remote builds in B18031: Diff 24703.Apr 6 2023, 3:51 AM

Closed by commit rCOMM49c2a6e22823: [web-db] extract worker write requests to a separate condition (authored by kamil). · Explain WhyApr 6 2023, 3:59 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM49c2a6e22823: [web-db] extract worker write requests to a separate condition.

kamil removed a child revision: D7289: [web-db] improve persisting lifecycle.Apr 27 2023, 6:48 AM

kamil added a child revision: D7289: [web-db] improve persisting lifecycle.May 8 2023, 2:42 AM

kamil removed a child revision: D7289: [web-db] improve persisting lifecycle.May 8 2023, 2:44 AM

kamil added a child revision: D7289: [web-db] improve persisting lifecycle.May 11 2023, 3:26 AM

Revision Contents
Changeset List

Path

Size

web/

database/

worker/

15 lines

types/

worker-types.js

7 lines

Diff 24705

web/database/worker/db-worker.js

Loading...

web/types/worker-types.js

Loading...