-
Notifications
You must be signed in to change notification settings - Fork 28.2k
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-17732][SQL] Revert ALTER TABLE DROP PARTITION should support comparators #16036
Conversation
…comparators" This reverts commit 3ce057d.
LGTM pending jenkins. @dongjoon-hyun lets merge the implementation of |
I see. @cloud-fan and @hvanhovell . |
Test build #69248 has finished for PR 16036 at commit
|
LGTM. Merging to master (2.1 was already reverted). Thanks! |
@hvanhovell , could you give me some direction on #15987 before updating? |
…omparators ## What changes were proposed in this pull request? apache#15704 will fail if we use int literal in `DROP PARTITION`, and we have reverted it in branch-2.1. This PR reverts it in master branch, and add a regression test for it, to make sure the master branch is healthy. ## How was this patch tested? new regression test Author: Wenchen Fan <wenchen@databricks.com> Closes apache#16036 from cloud-fan/revert.
…omparators ## What changes were proposed in this pull request? apache#15704 will fail if we use int literal in `DROP PARTITION`, and we have reverted it in branch-2.1. This PR reverts it in master branch, and add a regression test for it, to make sure the master branch is healthy. ## How was this patch tested? new regression test Author: Wenchen Fan <wenchen@databricks.com> Closes apache#16036 from cloud-fan/revert.
What changes were proposed in this pull request?
#15704 will fail if we use int literal in
DROP PARTITION
, and we have reverted it in branch-2.1.This PR reverts it in master branch, and add a regression test for it, to make sure the master branch is healthy.
How was this patch tested?
new regression test