Author: jvs
Date: Sun Nov 6 21:22:22 2011
New Revision: 1198566
URL: http://svn.apache.org/viewvc?rev=1198566&view=rev
Log:
HIVE-11. better error code from Hive describe command
(Ashutosh Chauhan via jvs)
Added:
hive/trunk/ql/src/test/queries/clientpositive/desc_non_existent_tbl.q
hive/trunk/ql/src/test/results/clientpositive/desc_non_existent_tbl.q.out
Modified:
hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
hive/trunk/ql/src/test/queries/clientnegative/external1.q
hive/trunk/ql/src/test/queries/clientnegative/external2.q
hive/trunk/ql/src/test/results/clientnegative/external1.q.out
hive/trunk/ql/src/test/results/clientnegative/external2.q.out
Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
URL:
http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java?rev=1198566&r1=1198565&r2=1198566&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
(original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java Sun Nov
6 21:22:22 2011
@@ -2516,7 +2516,7 @@ public class DDLTask extends Task<DDLWor
Path resFile = new Path(descTbl.getResFile());
if (tbl == null) {
FileSystem fs = resFile.getFileSystem(conf);
- outStream = (DataOutput) fs.open(resFile);
+ outStream = fs.create(resFile);
String errMsg = "Table " + tableName + " does not exist";
outStream.write(errMsg.getBytes("UTF-8"));
((FSDataOutputStream) outStream).close();
@@ -2527,7 +2527,7 @@ public class DDLTask extends Task<DDLWor
part = db.getPartition(tbl, descTbl.getPartSpec(), false);
if (part == null) {
FileSystem fs = resFile.getFileSystem(conf);
- outStream = (DataOutput) fs.open(resFile);
+ outStream = fs.create(resFile);
String errMsg = "Partition " + descTbl.getPartSpec() + " for table "
+ tableName + " does not exist";
outStream.write(errMsg.getBytes("UTF-8"));
Modified: hive/trunk/ql/src/test/queries/clientnegative/external1.q
URL:
http://svn.apache.org/viewvc/hive/trunk/ql/src/test/queries/clientnegative/external1.q?rev=1198566&r1=1198565&r2=1198566&view=diff
==============================================================================
--- hive/trunk/ql/src/test/queries/clientnegative/external1.q (original)
+++ hive/trunk/ql/src/test/queries/clientnegative/external1.q Sun Nov 6
21:22:22 2011
@@ -1,4 +1,3 @@
-set hive.cli.errors.ignore=true;
create external table external1(a int, b int) location
'invalidscheme://data.s3ndemo.hive/kv';
describe external1;
Modified: hive/trunk/ql/src/test/queries/clientnegative/external2.q
URL:
http://svn.apache.org/viewvc/hive/trunk/ql/src/test/queries/clientnegative/external2.q?rev=1198566&r1=1198565&r2=1198566&view=diff
==============================================================================
--- hive/trunk/ql/src/test/queries/clientnegative/external2.q (original)
+++ hive/trunk/ql/src/test/queries/clientnegative/external2.q Sun Nov 6
21:22:22 2011
@@ -1,4 +1,3 @@
-set hive.cli.errors.ignore=true;
create external table external2(a int, b int) partitioned by (ds string);
alter table external2 add partition (ds='2008-01-01') location
'invalidscheme://data.s3ndemo.hive/pkv/2008-01-01';
Added: hive/trunk/ql/src/test/queries/clientpositive/desc_non_existent_tbl.q
URL:
http://svn.apache.org/viewvc/hive/trunk/ql/src/test/queries/clientpositive/desc_non_existent_tbl.q?rev=1198566&view=auto
==============================================================================
--- hive/trunk/ql/src/test/queries/clientpositive/desc_non_existent_tbl.q
(added)
+++ hive/trunk/ql/src/test/queries/clientpositive/desc_non_existent_tbl.q Sun
Nov 6 21:22:22 2011
@@ -0,0 +1 @@
+desc nonexistenttbl;
Modified: hive/trunk/ql/src/test/results/clientnegative/external1.q.out
URL:
http://svn.apache.org/viewvc/hive/trunk/ql/src/test/results/clientnegative/external1.q.out?rev=1198566&r1=1198565&r2=1198566&view=diff
==============================================================================
--- hive/trunk/ql/src/test/results/clientnegative/external1.q.out (original)
+++ hive/trunk/ql/src/test/results/clientnegative/external1.q.out Sun Nov 6
21:22:22 2011
@@ -2,6 +2,3 @@ PREHOOK: query: create external table ex
PREHOOK: type: CREATETABLE
FAILED: Error in metadata: MetaException(message:Got exception:
java.io.IOException No FileSystem for scheme: invalidscheme)
FAILED: Execution Error, return code 1 from
org.apache.hadoop.hive.ql.exec.DDLTask
-PREHOOK: query: describe external1
-PREHOOK: type: DESCTABLE
-FAILED: Execution Error, return code 1 from
org.apache.hadoop.hive.ql.exec.DDLTask
Modified: hive/trunk/ql/src/test/results/clientnegative/external2.q.out
URL:
http://svn.apache.org/viewvc/hive/trunk/ql/src/test/results/clientnegative/external2.q.out?rev=1198566&r1=1198565&r2=1198566&view=diff
==============================================================================
--- hive/trunk/ql/src/test/results/clientnegative/external2.q.out (original)
+++ hive/trunk/ql/src/test/results/clientnegative/external2.q.out Sun Nov 6
21:22:22 2011
@@ -8,6 +8,3 @@ PREHOOK: type: ALTERTABLE_ADDPARTS
PREHOOK: Input: default@external2
FAILED: Error in metadata: MetaException(message:Got exception:
java.io.IOException No FileSystem for scheme: invalidscheme)
FAILED: Execution Error, return code 1 from
org.apache.hadoop.hive.ql.exec.DDLTask
-PREHOOK: query: describe external2 partition (ds='2008-01-01')
-PREHOOK: type: DESCTABLE
-FAILED: Execution Error, return code 1 from
org.apache.hadoop.hive.ql.exec.DDLTask
Added: hive/trunk/ql/src/test/results/clientpositive/desc_non_existent_tbl.q.out
URL:
http://svn.apache.org/viewvc/hive/trunk/ql/src/test/results/clientpositive/desc_non_existent_tbl.q.out?rev=1198566&view=auto
==============================================================================
--- hive/trunk/ql/src/test/results/clientpositive/desc_non_existent_tbl.q.out
(added)
+++ hive/trunk/ql/src/test/results/clientpositive/desc_non_existent_tbl.q.out
Sun Nov 6 21:22:22 2011
@@ -0,0 +1,5 @@
+PREHOOK: query: desc nonexistenttbl
+PREHOOK: type: DESCTABLE
+POSTHOOK: query: desc nonexistenttbl
+POSTHOOK: type: DESCTABLE
+Table nonexistenttbl does not exist