Skip to content

Unroll TaskState into two new SQL table (write-only for now). - #9485

Open
isoos wants to merge 6 commits into
dart-lang:masterfrom
isoos:task-state
Open

Unroll TaskState into two new SQL table (write-only for now).#9485
isoos wants to merge 6 commits into
dart-lang:masterfrom
isoos:task-state

Conversation

@isoos

@isoos isoos commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@isoos
isoos requested a review from jonasfj July 16, 2026 15:15
@isoos
isoos force-pushed the task-state branch 2 times, most recently from a07678e to 212cf41 Compare August 10, 2026 09:55
@isoos

isoos commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator Author

PTAL. I've updated the naming of the fields + columns:

  • I think it helps if timestamps are always ending with At.
  • We also need to synchronize the secretToken+token names.
  • The worker* prefix is debatable, I find it better to read.
  • I've also considered renaming attempts to attemptCount still undecided on it.

Comment thread app/lib/database/schema.task.dart
@isoos
isoos force-pushed the task-state branch 3 times, most recently from 9580307 to 65c2a3f Compare August 14, 2026 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants