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




sentry-hdfs/sentry-hdfs-service/pom.xml
Line 108 (original), 109 (patched)
<https://reviews.apache.org/r/67093/#comment285788>

    You have the same dependency at link 76. Do you need to declare it again?



sentry-provider/sentry-provider-db/src/main/resources/002-SENTRY-339.mysql.sql
Line 13 (original)
<https://reviews.apache.org/r/67093/#comment285789>

    these files only have change in empty lines, and are not part of your 
change. Can you revert them?



sentry-provider/sentry-provider-db/src/main/resources/010-SENTRY-2210.oracle.sql
Line 3 (original)
<https://reviews.apache.org/r/67093/#comment285790>

    You are reverting my changes. Can you revert your changes on all *.sql?


- Na Li


On May 21, 2018, 3:01 p.m., Anthony Young-Garner wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/67093/
> -----------------------------------------------------------
> 
> (Updated May 21, 2018, 3:01 p.m.)
> 
> 
> Review request for sentry.
> 
> 
> Bugs: SENTRY-2208
>     https://issues.apache.org/jira/browse/SENTRY-2208
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> Refactored the SentryService class and its dependencies from 
> sentry-provider/sentry-provider-db into a new 
> sentry-service/sentry-service-server module. In addition, refactored 
> SentryServiceClientFactory into a new sentry-service/sentry-service-client 
> module.
> 
> Work is based on previously completed work on SENTRY-1205.
> 
> NOTE: The diff looks large, but the overwhelming majority (9 pages) of the 
> diff consists of file moves (from sentry-provider-db to sentry-service-server)
> 
> https://issues.apache.org/jira/browse/SENTRY-2208
> 
> 
> Diffs
> -----
> 
>   sentry-hdfs/sentry-hdfs-service/pom.xml 
> 50a451d917eb7428cd71ab7cedd9c4363111c166 
>   sentry-provider/sentry-provider-db/pom.xml 
> 48a187ae54afd435c1f3d4aa57b0d696cd634fbe 
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/api/generic/thrift/SentryGenericPolicyProcessor.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/api/generic/thrift/SentryGenericPolicyProcessorFactory.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/api/service/thrift/ConfServlet.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/api/service/thrift/LogLevelServlet.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/api/service/thrift/PubSubServlet.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/api/service/thrift/SentryAdminServlet.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/api/service/thrift/SentryAuthFilter.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/api/service/thrift/SentryHealthCheckServletContextListener.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/api/service/thrift/SentryMetrics.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/api/service/thrift/SentryMetricsServletContextListener.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyStoreProcessor.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/api/service/thrift/SentryPolicyStoreProcessorFactory.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/api/service/thrift/SentryWebServer.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/SentryPolicyStorePlugin.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/persistent/DelegateSentryStore.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/persistent/PrivilegeObject.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/persistent/PrivilegeOperatePersistence.java
>  876ee146ab31aaa82b83e88585c45bb2deabc0d2 
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/persistent/SentryStoreLayer.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools/GenericPrivilegeConverter.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools/TSentryPrivilegeConverter.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/log/appender/AuditLoggerTestAppender.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/log/appender/RollingFileWithoutDeleteAppender.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/log/entity/AuditMetadataLogEntity.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/log/entity/DBAuditMetadataLogEntity.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/log/entity/GMAuditMetadataLogEntity.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/log/entity/JsonLogEntity.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/log/entity/JsonLogEntityFactory.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/log/util/CommandUtil.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/log/util/Constants.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MAuthzPathsMapping.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MAuthzPathsSnapshotId.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MPath.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryChange.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryGMPrivilege.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryGroup.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryHmsNotification.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPathChange.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPermChange.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryPrivilege.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryRole.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryUser.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryUtil.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/MSentryVersion.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/model/package.jdo
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/CounterWait.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/DeltaTransactionBlock.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/FixedJsonInstanceSerializer.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/HAContext.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/HMSFollower.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/LeaderStatusMonitor.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/NotificationProcessor.java
>  228d37c6f226fe94fc243fe9b6ec95feb696e4bd 
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/PathsImage.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/PermissionsImage.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/QueryParamBuilder.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
>  56c506b6331fea52208e9816438125db48ecd541 
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStoreSchemaInfo.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/TransactionBlock.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/TransactionManager.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/FullUpdateInitializer.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/FullUpdateModifier.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/GSSCallback.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSClient.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollowerState.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HiveConnectionFactory.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HiveNotificationFetcher.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HiveSimpleConnectionFactory.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/JaasConfiguration.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/KerberosConfiguration.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ProcessorFactory.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryHMSClient.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryKerberosContext.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryServiceClientFactory.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryServiceFactory.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryServiceState.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryState.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryStateBank.java
>   
>   
> sentry-provider/sentry-provider-db/src/main/resources/002-SENTRY-339.mysql.sql
>  cd4ec7c8400b56dd29c09c86b94c2591a36ef8d2 
>   
> sentry-provider/sentry-provider-db/src/main/resources/002-SENTRY-339.oracle.sql
>  f5f596d1e31c53ba53d831c5728e6e0af89b6451 
>   
> sentry-provider/sentry-provider-db/src/main/resources/002-SENTRY-339.postgres.sql
>  458e4477d46a8442c66537f3f21c429d03a2fa38 
>   
> sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.derby.sql
>  c038b8166174d860001ec21bca921613386964d6 
>   
> sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.mysql.sql
>  920737f135ae178804bb42e6d45d1b7ca0bcfddc 
>   
> sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.oracle.sql
>  412bc4557f0d41b01f38b8c3aab0d33e4b25e406 
>   
> sentry-provider/sentry-provider-db/src/main/resources/005-SENTRY-398.postgres.sql
>  e9e1655722206a291579b4f294099bd627e5cf06 
>   
> sentry-provider/sentry-provider-db/src/main/resources/010-SENTRY-2210.derby.sql
>  5c550902b88e1b475b128e5312b70475e9182d86 
>   
> sentry-provider/sentry-provider-db/src/main/resources/010-SENTRY-2210.oracle.sql
>  f4ba8d5e6bd5a0a06155b460e488b197fe76aa24 
>   
> sentry-provider/sentry-provider-db/src/main/resources/011-SENTRY-2154.derby.sql
>  e042ec4d73df2026cca1afcade27ddd994a797a9 
>   
> sentry-provider/sentry-provider-db/src/main/resources/011-SENTRY-2154.mysql.sql
>  db1201948807135570c88b37d2e1193cef7fb2eb 
>   
> sentry-provider/sentry-provider-db/src/main/resources/011-SENTRY-2154.oracle.sql
>  9852ac52a71c81915b25255819961bcb88f65ceb 
>   
> sentry-provider/sentry-provider-db/src/main/resources/011-SENTRY-2154.postgres.sql
>  a57e4f2f477e70d535401a6cbcdeaefeff9c7812 
>   sentry-provider/sentry-provider-db/src/main/resources/sentry-db2-2.1.0.sql 
> f2a109e0b0c74d805b42a872be38c9ce6fabfb94 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-derby-1.8.0.sql 
> d522026e18e6c2532487e9d4a988e408e6736008 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-derby-2.1.0.sql 
> b23e782c43efd9f3dfd946d9ac923b24e318d794 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-mysql-1.8.0.sql 
> d27d7b93cd3fb50d54c14c4eddee1ce59c785b13 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-mysql-2.1.0.sql 
> 2630e01bc0c8c8bfa5a1028a37fe0c4d41601607 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-1.4.0.sql 
> 363590edb0b5c0264799d90145c824cbff3b1679 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-1.5.0.sql 
> fe8e93c5b6b9f928597265583b0fdcd0271bc83a 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-1.6.0.sql 
> 3a22335503e835aa999c7b11dde6f5ee83b9d81a 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-1.7.0.sql 
> ae9cd0626e4f25077f06868e235122197ec720a7 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-oracle-2.1.0.sql 
> 847a359be895b4c096ab6dfb1323bcc9be52217a 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-postgres-2.1.0.sql
>  97a666ae5b38ec7279a6f9c8ce099be856427282 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-db2-2.0.0-to-2.1.0.sql
>  d6be0d611b77884fceb853195d43d68c3947c40a 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-derby-2.0.0-to-2.1.0.sql
>  0ea7f791ee7e74acb336a8aadead25df8b541f46 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-mysql-1.4.0-to-1.5.0.sql
>  a7bc9d00fca7137007a87eb135f0722c9fded424 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-mysql-2.0.0-to-2.1.0.sql
>  d679da8684c5b2b84e9b645728aeba7d4eaa10a9 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-oracle-2.0.0-to-2.1.0.sql
>  4a1d7c41715cd41073e8b65c706cf6b51d6de43c 
>   
> sentry-provider/sentry-provider-db/src/main/resources/sentry-upgrade-postgres-2.0.0-to-2.1.0.sql
>  4507d087efb06e96108a7cdd9c93e874e9b8b18f 
>   sentry-provider/sentry-provider-db/src/main/webapp/SentryService.html  
>   
> sentry-provider/sentry-provider-db/src/main/webapp/css/bootstrap-theme.min.css
>   
>   sentry-provider/sentry-provider-db/src/main/webapp/css/bootstrap.min.css  
>   sentry-provider/sentry-provider-db/src/main/webapp/css/sentry.css  
>   sentry-provider/sentry-provider-db/src/main/webapp/sentry.png  
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/generic/thrift/SentryGenericServiceIntegrationBase.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/generic/thrift/TestAuditLogForSentryGenericService.java
>  dea6152fd8226229a0aa5dd8b26cda10fc6bdc5b 
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/generic/thrift/TestSentryGenericPolicyProcessor.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/generic/thrift/TestSentryGenericServiceClient.java
>  2f72e88f3946e46b085b5bba05a8ac46ca0f178e 
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/generic/thrift/TestSentryGenericServiceIntegration.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/SentryMiniKdcTestcase.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestAuthorizingDDLAuditLogWithKerberos.java
>  fb412415a6bc3a82847b3cb5700fa4945ec0ac2f 
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestConnectionWithTicketTimeout.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestNotificationHandlerInvoker.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryPolicyServiceClient.java
>  e84593692cabde013694a9565c8ac7869ce09423 
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryPolicyStoreProcessor.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryServerForPoolWithoutKerberos.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryServerLogLevel.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryServerPubSub.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryServerWithoutKerberos.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryServiceClientPool.java
>  6c88955a56adaeca54edf8d68678f624c77a7698 
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryServiceFailureCase.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryServiceForPoolWithKerberos.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryServiceImportExport.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryServiceIntegration.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryServiceMetrics.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryServiceWithInvalidMsgSize.java
>  8ac63080f5770538bbe537aa61dbee15aab35f26 
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryServiceWithKerberos.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryWebServerWithKerberos.java
>  3cc9a9d1826eebc48e4ebf269b1eb7447c2921a4 
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryWebServerWithSSL.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/api/service/thrift/TestSentryWebServerWithoutSecurity.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/persistent/SentryStoreIntegrationBase.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/persistent/TestDelegateSentryStore.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/persistent/TestPrivilegeOperatePersistence.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/persistent/TestSentryGMPrivilege.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/persistent/TestSentryRole.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/log/appender/TestRollingFileWithoutDeleteAppender.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/log/entity/TestDbAuditMetadataLogEntity.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/log/entity/TestGMAuditMetadataLogEntity.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/log/entity/TestJsonLogEntityFactory.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/log/entity/TestJsonLogEntityFactoryGM.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/log/util/TestCommandUtil.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/model/TestMSentryUtil.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestCounterWait.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestHMSFollower.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestHMSFollowerSentryStoreIntegration.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestLeaderStatusMonitor.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestNotificationProcessor.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryPrivilege.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStore.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStoreImportExport.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryStoreToAuthorizable.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestSentryVersion.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/SentryServiceIntegrationBase.java
>  7e5e91dd84e806f2b35009298c3396ab3f632001 
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/SentryStateBankTestHelper.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/SentryWebMetricParser.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestFullUpdateInitializer.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestFullUpdateModifier.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestHiveNotificationFetcher.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestSentryHMSClient.java
>   
>   
> sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestSentryStateBank.java
>   
>   sentry-service/pom.xml 0388476f2e55dbc76262a4b0ddb597b816d8fb15 
>   sentry-service/sentry-service-client/pom.xml PRE-CREATION 
>   sentry-service/sentry-service-server/pom.xml PRE-CREATION 
>   
> sentry-service/sentry-service-server/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryWebServiceForAuthTypeNone.java
>  3d8bf960267c86c57e0acf9aa7572bf8aff67735 
>   sentry-service/sentry-service-server/src/test/resources/cacerts.jks 
> PRE-CREATION 
>   sentry-service/sentry-service-server/src/test/resources/keystore.jks 
> PRE-CREATION 
>   sentry-service/sentry-service-server/src/test/resources/log4j.properties 
> PRE-CREATION 
>   sentry-tests/sentry-tests-hive/pom.xml 
> 6816249da67a3fe9d0bc97217c5d8e8e184788a0 
>   sentry-tests/sentry-tests-kafka/pom.xml 
> e61f64db0c6787d09874235d061061add1b746d0 
>   sentry-tests/sentry-tests-solr/pom.xml 
> d9540a61f7394e3e316e8bd367c3d30aa7e66f14 
>   sentry-tests/sentry-tests-sqoop/pom.xml 
> eed8269d4e1b67fd622d539741e50d9984ae90c8 
>   sentry-tools/pom.xml b882c6f60dd0e3f33309e8e24ba1cab04fb7d86c 
> 
> 
> Diff: https://reviews.apache.org/r/67093/diff/5/
> 
> 
> Testing
> -------
> 
> mvn clean install
> 
> 
> Thanks,
> 
> Anthony Young-Garner
> 
>

Reply via email to