Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[IMPALA-13112] Remove Kafka Dependency Exclusion to Use KafkaAuditProvider #75

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dntjr8096
Copy link

Issue : https://issues.apache.org/jira/browse/IMPALA-13112

Testing Environment:
The changes were tested in the following environment:

  • Impala Version: 4.1.1
  • Ranger Version: 2.3.0

Results:
The tests confirmed that with the Kafka dependency exclusion removed:

Impala started successfully without any java.lang.NoClassDefFoundError errors related to javax.ws.rs.
Ranger's Kafka-based audit logging functioned correctly with the Kafka client included.
This change is expected to improve the integration between Impala and Ranger by enabling Kafka-based audit logging without reintroducing previous conflicts.

@dntjr8096 dntjr8096 force-pushed the master branch 3 times, most recently from 971d347 to a8d587c Compare May 29, 2024 08:35
@MikaelSmith
Copy link
Contributor

Thanks for your contribution! We don't use pull requests in Apache Impala, we use Gerrit instead: https://gerrit.cloudera.org/.

https://cwiki.apache.org/confluence/display/IMPALA/Using+Gerrit+to+submit+and+review+patches has a walkthrough for uploading patches for review.

…vider

Recent tests indicate that the latest version of `kafka-clients-2.8.1.jar`
used by Apache Ranger does not cause the previous JAX-RS conflicts.
Therefore, this exclusion is no longer necessary. Removing the exclusion
allows us to use the KafkaAuditProvider in Ranger for comprehensive audit
logging.

References:
1. [Commit a1f4a84](apache@a1f4a84)
2. [Commit f85dbff](apache@f85dbff)

Change-Id: I9a9cdab994552faa7b9d4c5da88d457dbd7b554d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants