-
Notifications
You must be signed in to change notification settings - Fork 28.9k
[SPARK-12968][SQL] Implement command to set current database #10916
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
Conversation
|
cc @hvanhovell for review. |
|
Test build #50071 has finished for PR 10916 at commit
|
|
retest this please. |
|
Test build #50079 has finished for PR 10916 at commit
|
|
Test build #50088 has finished for PR 10916 at commit
|
|
Test build #50090 has finished for PR 10916 at commit
|
|
Test build #50104 has finished for PR 10916 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.
Maybe a dumb idea: we could return Ok...
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.
Return OK will break hive compatibility test. I've tried in previous commits.
|
@viirya can you remove ur last commit? I can merge this. I'm redesigning catalog right now and we don't need to add that no-op commit here. |
|
@rxin ok. I will update this later (about 30 mins) when I am back to desk. |
|
Test build #50337 has finished for PR 10916 at commit
|
|
@rxin I've reverted last commit. |
|
Thanks - I've merged this. |
|
Test build #50350 has finished for PR 10916 at commit
|
JIRA: https://issues.apache.org/jira/browse/SPARK-12968
Implement command to set current database.