Re: Review Request 56403: SENTRY-1387 Add HDFS sync tests for drop partition for external/implicit locations

2017-02-08 Thread Nachiket Vaidya
> On Feb. 8, 2017, 6:59 p.m., Hao Hao wrote: > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/metastore/TestSentryListenerSentryDeserializer.java, > > line 232 > > > > > > Extra space? w

Re: Review Request 56403: SENTRY-1387 Add HDFS sync tests for drop partition for external/implicit locations

2017-02-08 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56403/#review164740 --- Fix it, then Ship it! sentry-tests/sentry-tests-hive/src/test/

Re: Review Request 56403: SENTRY-1387 Add HDFS sync tests for drop partition for external/implicit locations

2017-02-08 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56403/#review164700 --- Ship it! Ship It! - kalyan kumar kalvagadda On Feb. 7, 2017,

Re: Review Request 56403: SENTRY-1387 Add HDFS sync tests for drop partition for external/implicit locations

2017-02-07 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56403/#review164612 --- Ship it! Ship It! - Alexander Kolbasov On Feb. 7, 2017, 8:54

Review Request 56403: SENTRY-1387 Add HDFS sync tests for drop partition for external/implicit locations

2017-02-07 Thread Nachiket Vaidya
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56403/ --- Review request for sentry, Alexander Kolbasov, Hao Hao, and Vamsee Yarlagadda.