anfebladi opened a new pull request, #11292:
URL: https://github.com/apache/gravitino/pull/11292

   <!--
   [#8634] feat(catalog-hive): Hive catalog supports renaming table with new 
database name
   
   
   ### What changes were proposed in this pull request?
   
   added check to see if a new schema is passed when renaming a hive table and 
then move it to the new schema/database instead of just keeping it in the old 
one. also fixed some compilation issues with variables not being effectively 
final inside the hms client pool lambda and cleaned up the logic.
   
   ### Why are the changes needed?
   
   right now we don't support renaming hive tables into a different 
database/schema. this PR fixes that so we can move tables across databases in 
hive metastore during renames.
   
   Fix: #8634
   
   ### Does this PR introduce _any_ user-facing change?
   
   yes, users can now specify a new database name when renaming a table in the 
hive catalog using the rename api.
   
   ### How was this patch tested?
   
   ran existing tests in TestHiveTable and verified everything compiles and 
passes successfully.
   would be good to do further testing.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to