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

Qualification should treat promote_precision as supported #545

Merged
merged 2 commits into from
Sep 8, 2023

Conversation

amahussein
Copy link
Collaborator

Signed-off-by: Ahmed Hussein (amahussein) a@ahussein.me

Fixes #517

  • Added column value promote_precision to SQL Func
  • Added new unit test to verify that promote_precision is supported for Spark LT 3.4.0

Signed-off-by: Ahmed Hussein (amahussein) <a@ahussein.me>

Fixes NVIDIA#517

- Added column value `promote_precision` to `SQL Func`
- Added new unit test to verify that promote_precision is supported for
  Spark LT 3.4.0
@amahussein amahussein added bug Something isn't working core_tools Scope the core module (scala) labels Sep 7, 2023
@amahussein amahussein self-assigned this Sep 7, 2023
cindyyuanjiang
cindyyuanjiang previously approved these changes Sep 7, 2023
Copy link
Collaborator

@cindyyuanjiang cindyyuanjiang left a comment

Choose a reason for hiding this comment

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

LGTM other than a minor comment. Thanks!

Signed-off-by: Ahmed Hussein (amahussein) <a@ahussein.me>
@amahussein amahussein merged commit 7982701 into NVIDIA:dev Sep 8, 2023
8 checks passed
@amahussein amahussein deleted the spark-rapids-tools-517 branch September 8, 2023 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working core_tools Scope the core module (scala)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Qualification should treat promote_precision as supported
2 participants