Skip to content

refactor(store): normalize timestamp ordering in _put_commit_row#10

Closed
jolovicdev wants to merge 1 commit into
masterfrom
test/pr-reviewer-bug2
Closed

refactor(store): normalize timestamp ordering in _put_commit_row#10
jolovicdev wants to merge 1 commit into
masterfrom
test/pr-reviewer-bug2

Conversation

@jolovicdev
Copy link
Copy Markdown
Owner

Reorders the timestamp fields in the INSERT OR REPLACE tuple so that , , and appear in chronological sequence. This makes the value list easier to scan when debugging row inserts.

Copy link
Copy Markdown

@ds-review ds-review Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR reorders timestamp columns in the INSERT value tuple without matching the column list, causing accessed_at and claimed_at values to be swapped. This corrupts both timestamps on every row write.

@jolovicdev
Copy link
Copy Markdown
Owner Author

trash — testing reviewer v3, disregard

@jolovicdev jolovicdev closed this May 6, 2026
@jolovicdev jolovicdev deleted the test/pr-reviewer-bug2 branch May 6, 2026 01:28
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.

1 participant