You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by dm...@apache.org on 2019/11/01 14:03:08 UTC
[hive] branch master updated: HIVE-22428: Remove superfluous
'Failed to get database' WARN Logging in ObjectStore (David Mollitor,
reviewed by Miklos Gergely)
This is an automated email from the ASF dual-hosted git repository.
dmollitor pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git
The following commit(s) were added to refs/heads/master by this push:
new 8638894 HIVE-22428: Remove superfluous 'Failed to get database' WARN Logging in ObjectStore (David Mollitor, reviewed by Miklos Gergely)
8638894 is described below
commit 863889451ec838ed91ae4d80c03992a4eae4b482
Author: David Mollitor <dm...@apache.org>
AuthorDate: Fri Nov 1 10:00:15 2019 -0400
HIVE-22428: Remove superfluous 'Failed to get database' WARN Logging in ObjectStore (David Mollitor, reviewed by Miklos Gergely)
---
.../apache/hadoop/hive/metastore/ObjectStore.java | 54 +++++++++-------------
1 file changed, 21 insertions(+), 33 deletions(-)
diff --git a/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java b/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java
index e41c968..571eb86 100644
--- a/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java
+++ b/standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/ObjectStore.java
@@ -512,7 +512,7 @@ public class ObjectStore implements RawStore, Configurable {
@Override
public void createCatalog(Catalog cat) throws MetaException {
- LOG.debug("Creating catalog " + cat.getName());
+ LOG.debug("Creating catalog {}", cat);
boolean committed = false;
MCatalog mCat = catToMCat(cat);
try {
@@ -553,7 +553,7 @@ public class ObjectStore implements RawStore, Configurable {
@Override
public Catalog getCatalog(String catalogName) throws NoSuchObjectException, MetaException {
- LOG.debug("Fetching catalog " + catalogName);
+ LOG.debug("Fetching catalog {}", catalogName);
MCatalog mCat = getMCatalog(catalogName);
if (mCat == null) {
throw new NoSuchObjectException("No catalog " + catalogName);
@@ -585,7 +585,7 @@ public class ObjectStore implements RawStore, Configurable {
@Override
public void dropCatalog(String catalogName) throws NoSuchObjectException, MetaException {
- LOG.debug("Dropping catalog " + catalogName);
+ LOG.debug("Dropping catalog {}", catalogName);
boolean committed = false;
try {
openTransaction();
@@ -705,7 +705,7 @@ public class ObjectStore implements RawStore, Configurable {
ex = e;
}
if (db == null) {
- LOG.warn("Failed to get database {}.{}, returning NoSuchObjectException",
+ LOG.debug("Failed to get database {}.{}, returning NoSuchObjectException",
catalogName, name, ex);
throw new NoSuchObjectException(name + (ex == null ? "" : (": " + ex.getMessage())));
}
@@ -1526,7 +1526,7 @@ public class ObjectStore implements RawStore, Configurable {
}
if (LOG.isDebugEnabled()) {
- LOG.debug("getTableMeta with filter " + filterBuilder.toString() + " params: " +
+ LOG.debug("getTableMeta with filter " + filterBuilder + " params: " +
StringUtils.join(parameterVals, ", "));
}
// Add the fetch group here which retrieves the database object along with the MTable
@@ -1628,8 +1628,10 @@ public class ObjectStore implements RawStore, Configurable {
query.declareParameters(
"java.lang.String table, java.lang.String db, java.lang.String catname");
query.setUnique(true);
- LOG.debug("Executing getMTable for " +
- TableName.getQualified(catName, db, table));
+ if (LOG.isDebugEnabled()) {
+ LOG.debug("Executing getMTable for {}",
+ TableName.getQualified(catName, db, table));
+ }
mtbl = (MTable) query.execute(table, db, catName);
pm.retrieve(mtbl);
// Retrieving CD can be expensive and unnecessary, so do it only when required.
@@ -3623,9 +3625,7 @@ public class ObjectStore implements RawStore, Configurable {
message = ex.toString() + "; error building a better message: " + t.getMessage();
}
LOG.warn(message); // Don't log the exception, people just get confused.
- if (LOG.isDebugEnabled()) {
- LOG.debug("Full DirectSQL callstack for debugging (note: this is not an error)", ex);
- }
+ LOG.debug("Full DirectSQL callstack for debugging (not an error)", ex);
if (!allowJdo) {
if (ex instanceof MetaException) {
throw (MetaException)ex;
@@ -8679,7 +8679,8 @@ public class ObjectStore implements RawStore, Configurable {
committed = commitTransaction();
return result;
} finally {
- LOG.debug("Done executing getTableColumnStatistics with status : {}", committed);
+ LOG.debug("Done executing getTableColumnStatistics with status : {}",
+ committed);
rollbackAndCleanup(committed, query);
}
}
@@ -8785,7 +8786,8 @@ public class ObjectStore implements RawStore, Configurable {
committed = commitTransaction();
return result;
} finally {
- LOG.debug("Done executing getTableColumnStatistics with status : {}", committed);
+ LOG.debug("Done executing getTableColumnStatistics with status : {}",
+ committed);
rollbackAndCleanup(committed, query);
}
}
@@ -8913,8 +8915,9 @@ public class ObjectStore implements RawStore, Configurable {
if (!isCurrentStatsValidForTheQuery(part, part.getWriteId(), writeIdList, false)) {
String partName = Warehouse.makePartName(table.getPartitionKeys(), part.getValues());
- LOG.debug("The current metastore transactional partition column statistics for " + dbName
- + "." + tblName + "." + partName + " is not valid for the current query");
+ LOG.debug("The current metastore transactional partition column "
+ + "statistics for {}.{}.{} is not valid for the current query",
+ dbName, tblName, partName);
return null;
}
}
@@ -9615,27 +9618,12 @@ public class ObjectStore implements RawStore, Configurable {
return this.getMPartition(catName, dbName, tableName, name) != null;
}
- private void debugLog(String message) {
+ private void debugLog(final String message) {
if (LOG.isDebugEnabled()) {
- LOG.debug("{} {}", message, getCallStack());
+ LOG.debug("{}", message, new Exception());
}
}
- private static final int stackLimit = 3;
-
- private String getCallStack() {
- StackTraceElement[] stackTrace = Thread.currentThread().getStackTrace();
- int thislimit = Math.min(stackLimit, stackTrace.length);
- StringBuilder sb = new StringBuilder();
- sb.append(" at:");
- // Offset by 4 because the first 4 frames are just calls to get down here.
- for (int i = 4; i < thislimit + 4; i++) {
- sb.append("\n\t");
- sb.append(stackTrace[i].toString());
- }
- return sb.toString();
- }
-
private Function convertToFunction(MFunction mfunc) {
if (mfunc == null) {
return null;
@@ -11115,7 +11103,7 @@ public class ObjectStore implements RawStore, Configurable {
parameters.put("colType", type);
}
if (LOG.isDebugEnabled()) {
- LOG.debug("getSchemaVersionsByColumns going to execute query " + sql.toString());
+ LOG.debug("getSchemaVersionsByColumns going to execute query {}", sql);
LOG.debug("With parameters");
for (Map.Entry<String, String> p : parameters.entrySet()) {
LOG.debug(p.getKey() + " : " + p.getValue());
@@ -12435,7 +12423,7 @@ public class ObjectStore implements RawStore, Configurable {
@Override
public void addRuntimeStat(RuntimeStat stat) throws MetaException {
- LOG.debug("runtimeStat: " + stat);
+ LOG.debug("runtimeStat: {}", stat);
MRuntimeStat mStat = MRuntimeStat.fromThrift(stat);
boolean committed = false;
openTransaction();