ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chia-Ping Tsai <chia7...@gmail.com>
Subject Re: Review Request 73344: RANGER-3231 Ranger should use kafka Authorizer from KIP-504
Date Tue, 18 May 2021 04:16:51 GMT

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

(Updated 五月 18, 2021, 4:16 a.m.)


Review request for ranger.


Changes
-------

address comments from Madhan Neethiraj


Repository: ranger


Description
-------

As described in the KIP, `org.apache.kafka.server.authorizer.Authorizer` is an improvement
over `kafka.security.auth.Authorizer` and it's a pure Java interface (instead of Scala).
`kafka.security.auth.Authorizer` has been deprecated since December 2019 and it will be removed
in Apache Kafka 3.0 (roughly planned for July/August).
See the KIP for more details:
https://cwiki.apache.org/confluence/display/KAFKA/KIP-504+-+Add+new+Java+Authorizer+Interface


Diffs (updated)
-----

  plugin-kafka/src/main/java/org/apache/ranger/authorization/kafka/authorizer/RangerKafkaAuthorizer.java
2a1b812e0 
  ranger-kafka-plugin-shim/src/main/java/org/apache/ranger/authorization/kafka/authorizer/RangerKafkaAuthorizer.java
9d72ae0c8 


Diff: https://reviews.apache.org/r/73344/diff/2/

Changes: https://reviews.apache.org/r/73344/diff/1-2/


Testing
-------

run `mvn clean test` and all pass on my local.


File Attachments (updated)
----------------

RANGER-3231.v1.patch
  https://reviews.apache.org/media/uploaded/files/2021/05/18/4e2f190f-c871-4115-b554-0e6041a5a5a6__RANGER-3231.v1.patch


Thanks,

Chia-Ping Tsai


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message