You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by Sergio Pena <se...@cloudera.com> on 2017/09/08 21:26:26 UTC
Review Request 62194: SENTRY-1933: hive-authz2 build fails because
SentryJSONAlterPartitionMessage is not compatible
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62194/
-----------------------------------------------------------
Review request for sentry, Alexander Kolbasov and Na Li.
Bugs: sentry-1933
https://issues.apache.org/jira/browse/sentry-1933
Repository: sentry
Description
-------
The patch adds a new constructor visible for testing only to allow the hive-authz1 and hive-authz2 tests to compile.
Diffs
-----
sentry-binding/sentry-binding-hive-follower-v2/src/main/java/org/apache/sentry/binding/metastore/messaging/json/SentryJSONAlterPartitionMessage.java 64c15158b83d440577bcd2b717253ef9266d9b10
sentry-binding/sentry-binding-hive-follower/src/main/java/org/apache/sentry/binding/metastore/messaging/json/SentryJSONAlterPartitionMessage.java b29d727561ab0d2645c3c02fb118ca5c35508575
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestFullUpdateModifier.java a8fcbf8f36482661bb40adf481105af4e4438689
sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java e9ae6a1a503192c7fe6e6076913fdbc41957e56c
Diff: https://reviews.apache.org/r/62194/diff/1/
Testing
-------
Thanks,
Sergio Pena
Re: Review Request 62194: SENTRY-1933: hive-authz2 build fails because
SentryJSONAlterPartitionMessage is not compatible
Posted by Alexander Kolbasov <ak...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62194/#review185034
-----------------------------------------------------------
Ship it!
Ship It!
- Alexander Kolbasov
On Sept. 8, 2017, 9:26 p.m., Sergio Pena wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/62194/
> -----------------------------------------------------------
>
> (Updated Sept. 8, 2017, 9:26 p.m.)
>
>
> Review request for sentry, Alexander Kolbasov and Na Li.
>
>
> Bugs: sentry-1933
> https://issues.apache.org/jira/browse/sentry-1933
>
>
> Repository: sentry
>
>
> Description
> -------
>
> The patch adds a new constructor visible for testing only to allow the hive-authz1 and hive-authz2 tests to compile.
>
>
> Diffs
> -----
>
> sentry-binding/sentry-binding-hive-follower-v2/src/main/java/org/apache/sentry/binding/metastore/messaging/json/SentryJSONAlterPartitionMessage.java 64c15158b83d440577bcd2b717253ef9266d9b10
> sentry-binding/sentry-binding-hive-follower/src/main/java/org/apache/sentry/binding/metastore/messaging/json/SentryJSONAlterPartitionMessage.java b29d727561ab0d2645c3c02fb118ca5c35508575
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestFullUpdateModifier.java a8fcbf8f36482661bb40adf481105af4e4438689
> sentry-tests/sentry-tests-hive-v2/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegration.java e9ae6a1a503192c7fe6e6076913fdbc41957e56c
>
>
> Diff: https://reviews.apache.org/r/62194/diff/1/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Sergio Pena
>
>