Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add durable migrations#5250

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
amishler wants to merge7 commits intomain
base:main
Choose a base branch
Loading
fromalan/enable-durable
Open

Add durable migrations#5250

amishler wants to merge7 commits intomainfromalan/enable-durable

Conversation

@amishler
Copy link
Member

@amishleramishler commentedDec 17, 2025
edited by ellipsis-devbot
Loading

Add Postgres migrations to enable durable execution via thedurable crate for evaluations with top-k variant selection.


Important

Add durable migrations for evaluations with top-k variant selection using thedurable crate.

  • Migrations:
    • Add20251217184547_evaluations_topk.sql to createevaluations_topk queue for durable execution.
    • Updatemanual_run_postgres_migrations_with_url() inmod.rs to run durable migrations usingdurable::MIGRATOR.
  • Dependencies:
    • Adddurable crate toCargo.toml andCargo.lock.
  • Tests:
    • Update tests inexperimentation_queries.rs andrate_limit_queries.rs to usemanual_run_postgres_migrations_with_url() for setting up migrations.

This description was created byEllipsis forec94208. You cancustomize this summary. It will automatically update as commits are pushed.

ellipsis-dev[bot] reacted with rocket emoji
@amishleramishler marked this pull request as ready for reviewDecember 17, 2025 19:33
Aaron1011
Aaron1011 previously approved these changesDec 17, 2025
Aaron1011and others added4 commitsDecember 17, 2025 16:57
We now use 'manual_run_postgres_migrations_with_url' at the startof each test, since we have non-trivial dependencies betweenthe migrations in our various crates
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@Aaron1011Aaron1011Aaron1011 left review comments

At least 1 approving review is required to merge this pull request.

Assignees

@virajmehtavirajmehta

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@amishler@Aaron1011@virajmehta

[8]ページ先頭

©2009-2025 Movatter.jp