You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by br...@apache.org on 2013/10/28 15:15:23 UTC
svn commit: r1536361 -
/hive/trunk/service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java
Author: brock
Date: Mon Oct 28 14:15:23 2013
New Revision: 1536361
URL: http://svn.apache.org/r1536361
Log:
HIVE-5667 - ThriftCLIService log messages jumbled up (Vaibhav Gumashta via Brock Noland)
Modified:
hive/trunk/service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java
Modified: hive/trunk/service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java
URL: http://svn.apache.org/viewvc/hive/trunk/service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java?rev=1536361&r1=1536360&r2=1536361&view=diff
==============================================================================
--- hive/trunk/service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java (original)
+++ hive/trunk/service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java Mon Oct 28 14:15:23 2013
@@ -210,7 +210,7 @@ public abstract class ThriftCLIService e
resp.setOperationHandle(operationHandle.toTOperationHandle());
resp.setStatus(OK_STATUS);
} catch (Exception e) {
- LOG.warn("Error fetching results: ", e);
+ LOG.warn("Error executing statement: ", e);
resp.setStatus(HiveSQLException.toTStatus(e));
}
return resp;
@@ -224,7 +224,7 @@ public abstract class ThriftCLIService e
resp.setOperationHandle(operationHandle.toTOperationHandle());
resp.setStatus(OK_STATUS);
} catch (Exception e) {
- LOG.warn("Error executing statement: ", e);
+ LOG.warn("Error getting type info: ", e);
resp.setStatus(HiveSQLException.toTStatus(e));
}
return resp;
@@ -238,7 +238,7 @@ public abstract class ThriftCLIService e
resp.setOperationHandle(opHandle.toTOperationHandle());
resp.setStatus(OK_STATUS);
} catch (Exception e) {
- LOG.warn("Error getting type info: ", e);
+ LOG.warn("Error getting catalogs: ", e);
resp.setStatus(HiveSQLException.toTStatus(e));
}
return resp;
@@ -253,7 +253,7 @@ public abstract class ThriftCLIService e
resp.setOperationHandle(opHandle.toTOperationHandle());
resp.setStatus(OK_STATUS);
} catch (Exception e) {
- LOG.warn("Error getting catalogs: ", e);
+ LOG.warn("Error getting schemas: ", e);
resp.setStatus(HiveSQLException.toTStatus(e));
}
return resp;
@@ -269,7 +269,7 @@ public abstract class ThriftCLIService e
resp.setOperationHandle(opHandle.toTOperationHandle());
resp.setStatus(OK_STATUS);
} catch (Exception e) {
- LOG.warn("Error getting schemas: ", e);
+ LOG.warn("Error getting tables: ", e);
resp.setStatus(HiveSQLException.toTStatus(e));
}
return resp;
@@ -283,7 +283,7 @@ public abstract class ThriftCLIService e
resp.setOperationHandle(opHandle.toTOperationHandle());
resp.setStatus(OK_STATUS);
} catch (Exception e) {
- LOG.warn("Error getting tables: ", e);
+ LOG.warn("Error getting table types: ", e);
resp.setStatus(HiveSQLException.toTStatus(e));
}
return resp;
@@ -302,7 +302,7 @@ public abstract class ThriftCLIService e
resp.setOperationHandle(opHandle.toTOperationHandle());
resp.setStatus(OK_STATUS);
} catch (Exception e) {
- LOG.warn("Error getting table types: ", e);
+ LOG.warn("Error getting columns: ", e);
resp.setStatus(HiveSQLException.toTStatus(e));
}
return resp;
@@ -318,7 +318,7 @@ public abstract class ThriftCLIService e
resp.setOperationHandle(opHandle.toTOperationHandle());
resp.setStatus(OK_STATUS);
} catch (Exception e) {
- LOG.warn("Error getting columns: ", e);
+ LOG.warn("Error getting functions: ", e);
resp.setStatus(HiveSQLException.toTStatus(e));
}
return resp;
@@ -332,7 +332,7 @@ public abstract class ThriftCLIService e
resp.setOperationState(operationState.toTOperationState());
resp.setStatus(OK_STATUS);
} catch (Exception e) {
- LOG.warn("Error getting functions: ", e);
+ LOG.warn("Error getting operation status: ", e);
resp.setStatus(HiveSQLException.toTStatus(e));
}
return resp;
@@ -345,7 +345,7 @@ public abstract class ThriftCLIService e
cliService.cancelOperation(new OperationHandle(req.getOperationHandle()));
resp.setStatus(OK_STATUS);
} catch (Exception e) {
- LOG.warn("Error getting operation status: ", e);
+ LOG.warn("Error cancelling operation: ", e);
resp.setStatus(HiveSQLException.toTStatus(e));
}
return resp;
@@ -358,7 +358,7 @@ public abstract class ThriftCLIService e
cliService.closeOperation(new OperationHandle(req.getOperationHandle()));
resp.setStatus(OK_STATUS);
} catch (Exception e) {
- LOG.warn("Error canceling operation: ", e);
+ LOG.warn("Error closing operation: ", e);
resp.setStatus(HiveSQLException.toTStatus(e));
}
return resp;
@@ -373,7 +373,7 @@ public abstract class ThriftCLIService e
resp.setSchema(schema.toTTableSchema());
resp.setStatus(OK_STATUS);
} catch (Exception e) {
- LOG.warn("Error closing operation: ", e);
+ LOG.warn("Error getting result set metadata: ", e);
resp.setStatus(HiveSQLException.toTStatus(e));
}
return resp;
@@ -391,7 +391,7 @@ public abstract class ThriftCLIService e
resp.setHasMoreRows(false);
resp.setStatus(OK_STATUS);
} catch (Exception e) {
- LOG.warn("Error getting result set metadata: ", e);
+ LOG.warn("Error fetching results: ", e);
resp.setStatus(HiveSQLException.toTStatus(e));
}
return resp;