Skip to content

Revert "[WOMBAT-454] Let SQLAlchemy to assign the created/updated timeestamps. - #419

Merged
ntai-arxiv merged 1 commit into
developfrom
revert-cdf1444b
Jun 10, 2026
Merged

Revert "[WOMBAT-454] Let SQLAlchemy to assign the created/updated timeestamps.#419
ntai-arxiv merged 1 commit into
developfrom
revert-cdf1444b

Conversation

@ntai-arxiv

Copy link
Copy Markdown
Contributor

MySQL can handle it but sqlite3 cannot. Also it seems like some don't have the DDL defined with timestamp.

This reverts commit cdf1444.

…estamps. MySQL can handle it but sqlite3 cannot. Also it seems like some don't have the DDL defined with timestamp."

This reverts commit cdf1444.
@ntai-arxiv
ntai-arxiv requested review from bdc34 and kyokukou June 2, 2026 19:26
Comment thread arxiv/db/models.py

@bdc34 bdc34 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

See note. I'll approve the revert if that is what is wanted.

@ntai-arxiv

Copy link
Copy Markdown
Contributor Author

Yes. I will redo with something better.

@ntai-arxiv
ntai-arxiv requested a review from bdc34 June 3, 2026 23:52
@bdc34

bdc34 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Tai said in slack he is making a different PR

@bdc34 bdc34 closed this Jun 10, 2026
@ntai-arxiv ntai-arxiv reopened this Jun 10, 2026
@ntai-arxiv
ntai-arxiv merged commit 52eb7e1 into develop Jun 10, 2026
2 checks passed
@ntai-arxiv
ntai-arxiv deleted the revert-cdf1444b branch June 10, 2026 18:14
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.

2 participants