Skip to content
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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Cost model ORM implemenation #24

Merged
merged 28 commits into from
Nov 9, 2024
Merged

Cost model ORM implemenation #24

merged 28 commits into from
Nov 9, 2024

Conversation

unw9527
Copy link
Member

@unw9527 unw9527 commented Nov 7, 2024 •
edited
Loading

  • Add cost model related CRUD operations (with unit tests).
  • Add code to initialize all tables to support unit testing.

@unw9527 unw9527 mentioned this pull request Nov 7, 2024
Closed
@unw9527 unw9527 marked this pull request as ready for review November 9, 2024 16:21
@unw9527
Copy link
Member Author

unw9527 commented Nov 9, 2024

I think we can merge this? Otherwise, the branch might diverge too much. @xx01cyx @lanlou1554

@xx01cyx
Copy link
Member

xx01cyx commented Nov 9, 2024

I think we can merge this? Otherwise, the branch might diverge too much. @xx01cyx @lanlou1554

Go ahead

@connortsui20
Copy link
Member

Please let me know if you run into problems with the CI, it is relatively strict with dependencies and you'll need to run cargo build --locked from now on to adhere to the Cargo.lock file.

lanlou1554 and others added 25 commits November 9, 2024 13:19
@lanlou1554
Add implementation for create_new_epoch
304a042
@xx01cyx @lanlou1554
implement get_stats methods
99f36cf
@lanlou1554
Initial draft of update_stat
9f71c05
@lanlou1554
Small optimization
5dce3dd
@lanlou1554
Basic error handling
c05d664
@lanlou1554
Finish update_stats and store_expr_stats_mappings
3840673
@lanlou1554
Fix clippy
d05c493
@xx01cyx @lanlou1554
integrate w nullable columns
6d1f82e
@lanlou1554
Update test_create_new_epoch
2d12016
@unw9527 @lanlou1554
add: cost methods
72a07d1
@lanlou1554
add update_stats_from_catalog
586c51f
@lanlou1554
Add test_update_stats_from_catalog
959610b
@unw9527 @lanlou1554
feat: enable unit test with every table intialized
3cd2897
@lanlou1554
Fix clippy
ea154e6
@xx01cyx @lanlou1554
fix description
f56946c
@lanlou1554
Modify update_stat and add one related test
8cc4f0e
@xx01cyx @lanlou1554
refine test infra and track init.db
6371d61
@xx01cyx @lanlou1554
add more initial data into stat-related tables
fe891b0
@xx01cyx @lanlou1554
add new line at eof
2956971
@xx01cyx @lanlou1554
refine variant tag in init
6d70ad0
@xx01cyx @lanlou1554
use json for stat type
a82d6c4
@xx01cyx @lanlou1554
add test_get_stats_for_table
4103607
@xx01cyx @lanlou1554
minor fixes
7310fd2
@xx01cyx @lanlou1554
add test_get_stats_for_single_attr and test_get_stats_for_multiple_attrs
3c22282
@xx01cyx @lanlou1554
remove unused comments
5ad0c7a
@lanlou1554 lanlou1554 force-pushed the cost-model-ORM-impl-new branch from 52d290a to 7a8d4df Compare November 9, 2024 18:30
@lanlou1554 lanlou1554 merged commit 359e28a into main Nov 9, 2024
13 checks passed
@lanlou1554 lanlou1554 deleted the cost-model-ORM-impl-new branch November 9, 2024 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@unw9527 @xx01cyx @connortsui20 @lanlou1554