-
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-32560][SQL]Improve exception message at InsertIntoHiveTable.processInsert #29376
Conversation
update masterbranch
ok to test |
Looks okay if the tests pass. |
sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/InsertIntoHiveTable.scala
Outdated
Show resolved
Hide resolved
Test build #127162 has finished for PR 29376 at commit
|
Test build #127168 has finished for PR 29376 at commit
|
The tests passed at #29376 (comment). |
…rocessInsert ### What changes were proposed in this pull request? improve exception message ### Why are the changes needed? the before message lack of single quotes, we may improve it to keep consisent. ![image](https://user-images.githubusercontent.com/46367746/89595808-15bbc300-d888-11ea-9914-b05ea7b66461.png) ### Does this PR introduce _any_ user-facing change? NO ### How was this patch tested? No ,it is only improving the message. Closes #29376 from GuoPhilipse/improve-exception-message. Lead-authored-by: GuoPhilipse <46367746+GuoPhilipse@users.noreply.github.com> Co-authored-by: GuoPhilipse <guofei_ok@126.com> Signed-off-by: HyukjinKwon <gurwls223@apache.org> (cherry picked from commit aa4d3c1) Signed-off-by: HyukjinKwon <gurwls223@apache.org>
Merged to master, branch-3.0 and branch-2.4. |
…rocessInsert ### What changes were proposed in this pull request? improve exception message ### Why are the changes needed? the before message lack of single quotes, we may improve it to keep consisent. ![image](https://user-images.githubusercontent.com/46367746/89595808-15bbc300-d888-11ea-9914-b05ea7b66461.png) ### Does this PR introduce _any_ user-facing change? NO ### How was this patch tested? No ,it is only improving the message. Closes #29376 from GuoPhilipse/improve-exception-message. Lead-authored-by: GuoPhilipse <46367746+GuoPhilipse@users.noreply.github.com> Co-authored-by: GuoPhilipse <guofei_ok@126.com> Signed-off-by: HyukjinKwon <gurwls223@apache.org> (cherry picked from commit aa4d3c1) Signed-off-by: HyukjinKwon <gurwls223@apache.org>
cc @ScrapCodes and @zhengruifeng. This is very trivial but I cc'ed you guys since RC1 2.4 came out and you guys will probably be keen about the changes coming :-). |
Test build #127175 has finished for PR 29376 at commit
|
What changes were proposed in this pull request?
improve exception message
Why are the changes needed?
the before message lack of single quotes, we may improve it to keep consisent.
Does this PR introduce any user-facing change?
NO
How was this patch tested?
No ,it is only improving the message.