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

[FEA] Remove Speedup estimation from PluginTypeChecker #1343

Open
Tracked by #1221
amahussein opened this issue Sep 11, 2024 · 0 comments
Open
Tracked by #1221

[FEA] Remove Speedup estimation from PluginTypeChecker #1343

amahussein opened this issue Sep 11, 2024 · 0 comments
Assignees
Labels
core_tools Scope the core module (scala) feature request New feature or request

Comments

@amahussein
Copy link
Collaborator

Is your feature request related to a problem? Please describe.

After the static speedups have been removed. We should cleanup the code to get rid of the operator scores defined for each environment.
The two main classes affected by that change are

  • PluginTypeChecker and
  • SqlPlanParser
  • Qualification

Describe the solution you'd like
This can be done in 2 steps:

  1. Remove the operator score files and default the speedup to be 1.5 for all return.
  2. Refactor the qualification to eventually get rid of the code that calculates any speedups or GPU durations.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core_tools Scope the core module (scala) feature request New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant