bhaveshamre commented on code in PR #910:
URL: https://github.com/apache/ranger/pull/910#discussion_r3072315156


##########
kms/pom.xml:
##########
@@ -103,6 +103,16 @@
             <artifactId>jsr305</artifactId>
             <version>${jsr305.version}</version>
         </dependency>
+        <dependency>

Review Comment:
   We do not use failureaccess directly in our codebase; however, it is a 
required dependency used internally by Google Guava. It has been added 
explicitly to ensure it is available in the packaged runtime.
   
   This was necessary because, after integrating Google Cloud KMS, the runtime 
environment (Ranger/Hadoop) did not reliably resolve all required dependencies, 
leading to a NoClassDefFoundError for 
com/google/common/util/concurrent/internal/InternalFutureFailureAccess.
   
   Including this dependency explicitly ensures all required classes are 
present at runtime and prevents such classpath-related failures.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to