deniskuzZ commented on code in PR #6401:
URL: https://github.com/apache/hive/pull/6401#discussion_r3027595224
##########
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/jdbc/functions/OnRenameFunction.java:
##########
@@ -149,7 +149,12 @@ public Void execute(MultiDataSourceJdbcResource
jdbcResource) throws MetaExcepti
// caught and either swallowed or wrapped in MetaException. Also, only a
single test fails without this block:
//
org.apache.hadoop.hive.metastore.client.TestDatabases.testAlterDatabaseNotNullableFields
// It may worth investigate if this catch block is really needed.
- if (e.getMessage() != null && e.getMessage().contains("does not exist"))
{
+ String msg = e.getMessage();
Review Comment:
change is unrelated to the dependency upgrade
--
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]
