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

feat(cost-model): migration of compute cost parts #32

Closed
wants to merge 52 commits into from
Closed

feat(cost-model): migration of compute cost parts #32

wants to merge 52 commits into from

Conversation

lanlou1554
Copy link
Contributor

No description provided.

@lanlou1554 lanlou1554 force-pushed the cost-model-migration branch 3 times, most recently from 4090d0e to 9eed1c4 Compare November 15, 2024 17:51
lanlou1554 and others added 20 commits November 16, 2024 11:25
@lanlou1554
Partial migration of filter
c92a3af
@lanlou1554
Change col to attr
7b0158c
@xx01cyx @lanlou1554
implement cost computation for limit
81f8d50
@xx01cyx @lanlou1554
add author
2a5740e
@xx01cyx @lanlou1554
introduce ColumnCombValueStats
f7f6857
@xx01cyx @lanlou1554
refactor AttributeCombValueStats and introduce statistic-related data…
be430ac 
… structures
@lanlou1554
Change col to attr in filter
089cfef
@lanlou1554
Complete partial implementation of filter
69607f1
@lanlou1554
Add get_attribute_comb_stats
6518e00
@lanlou1554
Finish first draft version of filter functionality
59a8889
@lanlou1554
Add comment for the guideline of re-designing PredicateNode
5070a78
@xx01cyx @lanlou1554
introduce IdPred and make AttributeRefPred store table id and attr index
740ab11
@xx01cyx @lanlou1554
add get method for id pred and add comments
85cd0d1
@xx01cyx @lanlou1554
add check for derived column in AttributeRefPred
7775b88
@xx01cyx @lanlou1554
make get_attributes_comb_statistics return Option
b60c632
@xx01cyx @lanlou1554
implement agg cost computation
3646eca
@xx01cyx @lanlou1554
move filter-related constants to stats crate
db555ff
@xx01cyx @lanlou1554
fix clippy
64f4a10
@lanlou1554
Resolve the optional comb stats, remove table id in filter
cafd01c
@lanlou1554
Refactor filter implementation
5c5a40f
lanlou1554 and others added 23 commits November 16, 2024 23:19
@lanlou1554
Improve filter tests
d6e1825
@xx01cyx
refine test infra
082f0be
@xx01cyx
add test for cost model agg
e183f02
@xx01cyx
make all data types u64 instead of usize
0059141
@xx01cyx
merge main and resolve conflicts
303d73c
@xx01cyx
copy paste join cardinality calculation
ec0afa6
@xx01cyx
make join compile
6d50843
@xx01cyx
rename col -> attr
a4ff526
@xx01cyx
refactor join to not pass in logical props
0ba4132
@xx01cyx
make statistics f64 instead of u64
ab15f05
@xx01cyx
split join into multiple files
b682c73
@xx01cyx
reorganize join
5d73141
@xx01cyx
refine test infra
51f917d
@xx01cyx
add test infra for join
5197090
@xx01cyx
refine mock interface
68b2885
@xx01cyx
make CostModelStorageManagerImpl::get_attribute_info unimplemented
36b93b9
@xx01cyx
modify MemoExt interface
11a3a4e
@xx01cyx
rename AttrRefPred -> AttrIndexPred and revert back to initial design
8c4191f
@lanlou1554
Modify the tests of filter and agg
1569fc5
@xx01cyx
add join test
489ff48
@xx01cyx
pass group id to join and fix filter-related tests
be71afb
@xx01cyx
fix all join tests
624d040
@lanlou1554
Change filter controller name
f8a0e70
@lanlou1554
Copy link
Contributor Author

Have been merge by other PRs #40 #41 #42

@lanlou1554 lanlou1554 closed this Nov 20, 2024
@lanlou1554 lanlou1554 deleted the cost-model-migration branch November 20, 2024 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xx01cyx xx01cyx xx01cyx left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lanlou1554 @xx01cyx