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

[Algebrizer/Rewriter] make OptimizeOption.remove_from_ default to true #113

Open
zhouqingqing opened this issue Jun 9, 2020 · 1 comment
Open

[Algebrizer/Rewriter] make OptimizeOption.remove_from_ default to true #113

zhouqingqing opened this issue Jun 9, 2020 · 1 comment
Assignees
@pkommoju

Comments

@zhouqingqing
Copy link
Owner

This flag controls the behavior to remove redundant FROM (select * from (select * from a) b) after we fixed bugs with it, for example (aggregation from aggregation).
@zhouqingqing zhouqingqing changed the title make OptimizeOption.remove_from_ default to true [Algebrizer] make OptimizeOption.remove_from_ default to true Jun 9, 2020
@zhouqingqing zhouqingqing changed the title [Algebrizer] make OptimizeOption.remove_from_ default to true [Algebrizer/Rewriter] make OptimizeOption.remove_from_ default to true Jun 9, 2020
@pkommoju
Copy link
Collaborator

Removing remove_from is presenting problems when from contains union/intersect/except because outputs are not set for all except the first select.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pkommoju pkommoju

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@zhouqingqing @PerHChristensen @pkommoju