Github user gatorsmile commented on the pull request:

    https://github.com/apache/spark/pull/12385#issuecomment-210487137
  
    The test cases are wrong, based on the Hive definition. 
    - `Rename Partitions` only allows users to change the partition values, 
instead of partition column names.
    - `Add Partitions` only allows users to add new partitions with the exactly 
same partition columns defined in Table. 
    - `Alter Partition Set Location` also need to ensure users to use the same 
partitioning definition. 
    
    @andrewor14 @yhuai Do you want me to submit a separate PR to fix all the 
issues in the implementation of Alter Partition? In the `InMemoryCatalog`, we 
ignore all the above issues. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to