----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69352/ -----------------------------------------------------------
(Updated Dec. 7, 2018, 3:14 p.m.) Review request for sentry and Sergio Pena. Bugs: SENTRY-2452 https://issues.apache.org/jira/browse/SENTRY-2452 Repository: sentry Description ------- TSentryExportMappingDataRequest and TSentryImportMappingDataRequest which are used to send export and import requests to sentry server should be changed to be able to send a list authorizables for which permission information has to be exported/imported. These requests should accommodate source and target cluster information as well. Diffs (updated) ----- sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/utils/SentryConstants.java d8c1061d36ed0b92116f0b2dd5ec820ccb166818 sentry-hdfs/sentry-hdfs-common/src/gen/thrift/gen-javabean/org/apache/sentry/hdfs/service/thrift/TPrivilegePrincipalType.java 6eb85217b9229438b9adbd1546a408dc507f1645 sentry-service/sentry-service-api/src/gen/thrift/gen-javabean/org/apache/sentry/api/service/thrift/TSentryExportMappingDataRequest.java 13e57e04dd779825e2986b1e527f4e556271a162 sentry-service/sentry-service-api/src/gen/thrift/gen-javabean/org/apache/sentry/api/service/thrift/TSentryImportMappingDataRequest.java 21f3bdf5e304fe2f49684c999d0aa0e0362320de sentry-service/sentry-service-api/src/gen/thrift/gen-javabean/org/apache/sentry/api/service/thrift/TSentryPrivilegesResponse.java cea868f0524c25bf63cfb7c532829354a280df28 sentry-service/sentry-service-api/src/main/java/org/apache/sentry/api/common/SentryServiceUtil.java 83393a98df5e10b324f74dc12f10c20bc5f02651 sentry-service/sentry-service-api/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyServiceClientDefaultImpl.java 68d864cfbdf18057d87a65a04af8991292aadccf sentry-service/sentry-service-api/src/main/resources/sentry_policy_service.thrift 3364648dc88acbb8de1cc925fe8c512f34a4b064 sentry-service/sentry-service-api/src/test/java/org/apache/sentry/api/common/TestSentryServiceUtil.java 2dc0975f482f760c18c438c1418630a7ef6a4cbb sentry-service/sentry-service-server/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyStoreProcessor.java b9e3bf2921a0696da639e1b1bee5d83cf2b9cee0 sentry-service/sentry-service-server/src/test/java/org/apache/sentry/api/service/thrift/TestSentryServiceImportExport.java cf1fdab382034c8950da2613ef9b0cf1af912e33 Diff: https://reviews.apache.org/r/69352/diff/5/ Changes: https://reviews.apache.org/r/69352/diff/4-5/ Testing ------- Made sure that existiung tests pass. Thanks, kalyan kumar kalvagadda