Re: Review Request 63975: SENTRY-2066: DB name is not set for AlterTable

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63975/#review191659 --- Ship it! Ship It! - Arjun Mishra On Nov. 21, 2017, 9:27

Re: Review Request 63975: SENTRY-2066: DB name is not set for AlterTable

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63975/#review191656 --- Ship it! Ship It! - Sergio Pena On Nov. 21, 2017, 9:27

Re: Review Request 63975: SENTRY-2066: DB name is not set for AlterTable

> On Nov. 21, 2017, 8:09 p.m., Arjun Mishra wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java > > Line 265 (original), 265 (patched) > > > > > > Is

Re: Review Request 63975: SENTRY-2066: DB name is not set for AlterTable

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63975/ --- (Updated Nov. 21, 2017, 9:27 p.m.) Review request for sentry, Arjun Mishra,

Re: Review Request 63975: SENTRY-2066: DB name is not set for AlterTable

> On Nov. 21, 2017, 7:51 a.m., Alexander Kolbasov wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java > > Lines 283 (patched) > > > > > > Why do we

Re: Review Request 63975: SENTRY-2066: DB name is not set for AlterTable

> On Nov. 21, 2017, 7:51 a.m., Alexander Kolbasov wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java > > Lines 283 (patched) > > > > > > Why do we

Re: Review Request 63975: SENTRY-2066: DB name is not set for AlterTable

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63975/ --- (Updated Nov. 21, 2017, 5:08 p.m.) Review request for sentry, Arjun Mishra,

Re: Review Request 63975: SENTRY-2066: DB name is not set for AlterTable

> On Nov. 21, 2017, 7:30 a.m., Vadim Spector wrote: > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSentryOnFailureHookLoading.java > > Line 265 (original), 265 (patched) > >

Re: Review Request 63975: SENTRY-2066: DB name is not set for AlterTable

> On Nov. 21, 2017, 7:51 a.m., Alexander Kolbasov wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java > > Lines 283 (patched) > > > > > > Why do we

Re: Review Request 63975: SENTRY-2066: DB name is not set for AlterTable

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63975/#review191566 ---