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

Remove the redundant create_from_select_plan.py #1145

Open
1 of 2 tasks
xzdandy opened this issue Sep 17, 2023 · 0 comments
Open
1 of 2 tasks

Remove the redundant create_from_select_plan.py #1145

xzdandy opened this issue Sep 17, 2023 · 0 comments
Labels
Minor 💅 Polishing EVA is always nice

Comments

@xzdandy
Copy link
Collaborator

xzdandy commented Sep 17, 2023

Search before asking

  • I have searched the EvaDB issues and found no similar feature requests.

Description

The evadb/plan_nodes/create_from_select_plan.py is redundant. We can use the evadb/plan_nodes/create_plan.py instead across the codebase.

Use case

No response

Are you willing to submit a PR?

  • Yes I'd like to help by submitting a PR!
@xzdandy xzdandy added the Minor 💅 Polishing EVA is always nice label Sep 17, 2023
aayushacharya added a commit to aayushacharya/evadb that referenced this issue Feb 25, 2024
@aayushacharya
refactor: used create_plan across codebase, closes georgia-tech-db#…
3ddd92c 
…1145
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Minor 💅 Polishing EVA is always nice
Projects
EVA Public Roadmap ⚡🚀
Status: ToDo
Milestone
No milestone
Development

No branches or pull requests
1 participant
@xzdandy