----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47569/ -----------------------------------------------------------
Review request for sentry. Repository: sentry Description ------- As part of renewThread we are logging out the subject and relogging in. This is causing a client request to fail if it happens in this logout -login window. As only TGT needs renewal, we should never run the renewThread in Sentry given that Sentry never is a Kerberos Client to other Kerberos Services. Diffs ----- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryKerberosContext.java 93481cb32cb0d6743cfbc8dc7655ea6fcdf12d13 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java d8edf93771b29e1d2a0a533fd6a8b5772af1e6d0 sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java 42eb1bbe5ccbf0e28dcabc6655eb50dfbbd1ec23 sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericServiceIntegrationBase.java cec925b695142b409dfaf1db859e7eb267cd1f65 sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/thrift/TestAuditLogForSentryGenericService.java c3adacf52216b4ce331dbb92a06064ccd26aa043 sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestConnectionWithTicketTimeout.java e204099d81e10d3f0ce8aab2796d020df1712e2c sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceClientPool.java e5285bd0a760bdcc7fe27e6af96d80ebc51d7dad sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/SentryServiceIntegrationBase.java 14de0fac6a4a6cc25b36a65c4a7f02dd6fc135e9 Diff: https://reviews.apache.org/r/47569/diff/ Testing ------- Updated test TestConnectionWithTicketTimeout to make sure client is able to connect after max.life.time Thanks, Sravya Tirukkovalur