[GitHub] NamanRastogi opened a new pull request #3098: [CARBONDATA-3235] Fixed Alter Table Rename

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] NamanRastogi opened a new pull request #3098: [CARBONDATA-3235] Fixed Alter Table Rename

GitBox
NamanRastogi opened a new pull request #3098: [CARBONDATA-3235] Fixed Alter Table Rename
URL: https://github.com/apache/carbondata/pull/3098
 
 
   Mixed a mistake in [PR-2996](https://github.com/apache/carbondata/pull/2996).
   The newTableIdentifier should have been newIdentifier
   
   Fixed negative scenario:
   
   **Alter Table Rename Table Fail**
   
   Problem: _When tabe rename is success in hive, for failed in carbon data store, it would throw exception, but would not go back and undo rename in hive._
   
   Solution: _A flag to keep check if hive rename has already executed, and of the code breaks after hive rename is done, go back and undo the hive rename._
   
   
    - [x] Any interfaces changed?   ->  No
    - [x] Any backward compatibility impacted?  -> No
    - [x] Document update required?  ->  No
    - [x] Testing done  ->  Yes
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[hidden email]


With regards,
Apache Git Services