ashniku opened a new pull request, #6371: URL: https://github.com/apache/hive/pull/6371
…/DROP table What changes were proposed in this pull request? I have mofified "MetaStoreUtils.java", updated "hasCatalogName" to return true only if the database name both starts with the catalog marker (@) and contains the catalog-database separator (#). --> Why are the changes needed? The issue is caused by MetaStoreUtils.hasCatalogName incorrectly identifying database names that start with the '@' character as catalog-prepended names. This leads to a failure in MetaStoreUtils.parseDbName when it tries to split the name using the '#' separator, which is missing in regular database names starting with '@'. Proposed Changes [Metastore Common] [MODIFY] MetaStoreUtils.java Update hasCatalogName to return true only if the database name both starts with the catalog marker (@) and contains the catalog-database separator (#). Does this PR introduce any user-facing change? NO --> How was this patch tested? CREATE DATABASE @test; USE @test; CREATE TABLE testtable (c1 INT); ALTER TABLE testtable ADD COLUMNS (c2 INT); DROP TABLE testtable; DROP DATABASE @test; -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
