-
Notifications
You must be signed in to change notification settings - Fork 28.1k
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
[SPARK-4928][SQL] Fix: Operator '>,<,>=,<=' with decimal between different precision report error #3767
Conversation
Can one of the admins verify this patch? |
ok to test |
Test build #24729 has started for PR 3767 at commit
|
@@ -361,6 +361,21 @@ trait HiveTypeCoercion { | |||
DecimalType(min(p1 - s1, p2 - s2) + max(s1, s2), max(s1, s2)) | |||
) | |||
|
|||
// Cast is no need for logical operator |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NIT: "no need" => "not needed"
Could you please add a test case? Thanks! |
Test build #24729 has finished for PR 3767 at commit
|
Test PASSed. |
Test build #24733 has started for PR 3767 at commit
|
Test build #24736 has started for PR 3767 at commit
|
LGTM as long as Jenkins says yes. Thanks for working on this! |
Test build #24733 has finished for PR 3767 at commit
|
Test PASSed. |
Test build #24736 has finished for PR 3767 at commit
|
Test PASSed. |
Thanks! Merged to master. |
case operator with decimal between different precision, we need change them to unlimited