-
Notifications
You must be signed in to change notification settings - Fork 28k
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-28460][SQL][TEST][test-hadoop3.2] Port test from HIVE-11835 #25212
Conversation
This comment has been minimized.
This comment has been minimized.
Retest this please. |
Yes. This is a way to test this. I'm wondering if we can have a simpler test case for this. cc @gatorsmile |
This comment has been minimized.
This comment has been minimized.
...ompatibility/src/test/scala/org/apache/spark/sql/hive/execution/HiveCompatibilitySuite.scala
Show resolved
Hide resolved
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
retest this please |
Test build #107996 has finished for PR 25212 at commit
|
retest this please |
Test build #108004 has finished for PR 25212 at commit
|
Retest this please. |
Test build #108256 has finished for PR 25212 at commit
|
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.
+1, LGTM. Although this is a port from Hive, I'll merge this to the master
for Hive compatibility because we have several PRs against decimal behavior changes.
Thank you, @wangyum and @HyukjinKwon .
cc @mgaido91 , @gatorsmile , @cloud-fan , @felixcheung
What changes were proposed in this pull request?
HIVE-11835 fixed type
decimal(1,1)
reads 0.0, 0.00, etc from text file as NULL. We fixed this issue after upgrade the build-in Hive to 2.3.5. This PR port the test from HIVE-11835.Hive test result:
https://github.com/apache/hive/blob/release-2.3.5-rc0/ql/src/test/results/clientpositive/decimal_1_1.q.out#L67-L96
How was this patch tested?
N/A