mdayakar commented on code in PR #6854:
URL: https://github.com/apache/hive/pull/6854#discussion_r4203070580
##########
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/PartitionManagementTask.java:
##########
@@ -125,7 +126,8 @@ public void run() {
countDownLatch.await();
executorService.shutdownNow();
} catch (Exception e) {
- LOG.error("Exception while running partition discovery task for table:
" + qualifiedTableName, e);
+ String tableLogMsg = (qualifiedTableName != null) ? " for table: " +
qualifiedTableName : "";
Review Comment:
@soumyakanti3578 thanks for the review. When table name is available, the
log message should contain `" for table: <tablename"` so even if we declare
`qualifiedTableName` with empty string still the ternary check is required to
check empty. Also now the qualifiedTableName passed to `MsckThread` will
definitely have the table name as its part of the for loop.
--
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]
