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.

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

feat(merger): 定义中立的特征表达数据、定义工厂方法根据特征数据来获取具体的merger #222

Merged
merged 11 commits into from
Jun 11, 2024

Conversation

longyue0521
Copy link
Collaborator

采用map+位运算+工厂方法来解决 #197

Copy link
Contributor

@flycash flycash left a comment

Choose a reason for hiding this comment

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

在测试用例上,可以试试分叉的形态吗?比如说在讨论到聚合函数的时候,GROUP BY 是不是按照 sharding key 来进行的,也会直接印象到你选用哪个 merge。那么在这种情况下,你这个设计会如何运作?

@longyue0521 longyue0521 merged commit 9e6a3fc into ecodeclub:dev Jun 11, 2024
7 checks passed
@longyue0521 longyue0521 deleted the f_merger branch June 11, 2024 03:37
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