Re: Review Request 56118: DROP TABLE in hive doesn't Throw Error

2017-02-13 Thread Adam Szita
> On Feb. 10, 2017, 7:32 p.m., Vihang Karajgaonkar wrote: > > metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java, > > line 1679 > > > > > > I agree with Aihua here. As long as the table metadata

Re: Review Request 56118: DROP TABLE in hive doesn't Throw Error

2017-02-10 Thread Vihang Karajgaonkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56118/#review165150 --- metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore

Re: Review Request 56118: DROP TABLE in hive doesn't Throw Error

2017-02-10 Thread Aihua Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56118/#review165125 --- metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore

Re: Review Request 56118: DROP TABLE in hive doesn't Throw Error

2017-02-06 Thread Adam Szita
> On Feb. 3, 2017, 2:42 p.m., Aihua Xu wrote: > > metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java, > > line 1679 > > > > > > With this approach, I feel it will place Hive in worse situation s

Re: Review Request 56118: DROP TABLE in hive doesn't Throw Error

2017-02-03 Thread Aihua Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56118/#review164121 --- metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore

Re: Review Request 56118: DROP TABLE in hive doesn't Throw Error

2017-02-03 Thread Adam Szita
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56118/ --- (Updated Feb. 3, 2017, 2:22 p.m.) Review request for hive, Aihua Xu, Peter Vary

Review Request 56118: DROP TABLE in hive doesn't Throw Error

2017-01-31 Thread Adam Szita
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56118/ --- Review request for hive, Aihua Xu, Peter Vary, and Sergio Pena. Bugs: HIVE-1418