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

[Tech Request]: add projection support for some operators #19769

Open
2 tasks done
ouyuanning opened this issue Nov 4, 2024 · 0 comments
Open
2 tasks done

[Tech Request]: add projection support for some operators #19769

ouyuanning opened this issue Nov 4, 2024 · 0 comments
Assignees
Labels
kind/tech-request New feature or request priority/p0 Critical feature that should be implemented in this version
Milestone

Comments

@ouyuanning
Copy link
Contributor

Is there an existing issue for the same tech request?

  • I have checked the existing issues.

Does this tech request not affect user experience?

  • This tech request doesn't affect user experience.

What would you like to be added ?

之前曾言有给部分算子增加了Projection的支持,但是存在一些问题又去掉了。
可以继续补回来。
1、应该绝大部分算子都可以有projection
2、以前的处理感觉可以优化:
    1. compile的地方,应该没必要那么多if。可能直接在CompilePrjection增加上一个算子的引用,然后统一处理即可
    2. 调用的地方,不知道是否参考 ChildrenCall 再封装一个 输出的函数,让所有的输出都用那一个函数。因为后面可能还会再增加filter

Why is this needed ?

缩短pipeline的长度,减少算子间的调用链,提升性能

Additional information

No response

@ouyuanning ouyuanning added kind/tech-request New feature or request priority/p0 Critical feature that should be implemented in this version labels Nov 4, 2024
@ouyuanning ouyuanning added this to the 2.0.1 milestone Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/tech-request New feature or request priority/p0 Critical feature that should be implemented in this version
Projects
None yet
Development

No branches or pull requests

2 participants