You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by ma...@apache.org on 2015/02/25 04:51:40 UTC
spark git commit: [SPARK-5286][SQL] SPARK-5286 followup
Repository: spark
Updated Branches:
refs/heads/master 922b43b3c -> 769e092bd
[SPARK-5286][SQL] SPARK-5286 followup
https://issues.apache.org/jira/browse/SPARK-5286
Author: Yin Huai <yh...@databricks.com>
Closes #4755 from yhuai/SPARK-5286-throwable and squashes the following commits:
4c0c450 [Yin Huai] Catch Throwable instead of Exception.
Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/769e092b
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/769e092b
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/769e092b
Branch: refs/heads/master
Commit: 769e092bdc51582372093f76dbaece27149cc4ea
Parents: 922b43b
Author: Yin Huai <yh...@databricks.com>
Authored: Tue Feb 24 19:51:36 2015 -0800
Committer: Michael Armbrust <mi...@databricks.com>
Committed: Tue Feb 24 19:51:36 2015 -0800
----------------------------------------------------------------------
.../scala/org/apache/spark/sql/hive/execution/commands.scala | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/spark/blob/769e092b/sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/commands.scala
----------------------------------------------------------------------
diff --git a/sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/commands.scala b/sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/commands.scala
index c88d0e6..9934a5d 100644
--- a/sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/commands.scala
+++ b/sql/hive/src/main/scala/org/apache/spark/sql/hive/execution/commands.scala
@@ -63,10 +63,10 @@ case class DropTable(
} catch {
// This table's metadata is not in
case _: org.apache.hadoop.hive.ql.metadata.InvalidTableException =>
- // Other exceptions can be caused by users providing wrong parameters in OPTIONS
+ // Other Throwables can be caused by users providing wrong parameters in OPTIONS
// (e.g. invalid paths). We catch it and log a warning message.
- // Users should be able to drop such kinds of tables regardless if there is an exception.
- case e: Exception => log.warn(s"${e.getMessage}")
+ // Users should be able to drop such kinds of tables regardless if there is an error.
+ case e: Throwable => log.warn(s"${e.getMessage}")
}
hiveContext.invalidateTable(tableName)
hiveContext.runSqlHive(s"DROP TABLE $ifExistsClause$tableName")
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org