----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74942/#review226339 -----------------------------------------------------------
Kirby - the service-def for Pegasus looks good. However, the plugin implementation is not included in this review. Can you please review and update? agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java Lines 102 (patched) <https://reviews.apache.org/r/74942/#comment314580> RangerServiceKudu => RangerServicePegasus - Madhan Neethiraj On March 20, 2024, 11:05 a.m., Kirby Zhou wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74942/ > ----------------------------------------------------------- > > (Updated March 20, 2024, 11:05 a.m.) > > > Review request for ranger, Bhavik Bavishi and Madhan Neethiraj. > > > Bugs: RANGER-3831 > https://issues.apache.org/jira/browse/RANGER-3831 > > > Repository: ranger > > > Description > ------- > > Apache Pegasus is A horizontally scalable, strongly consistent and > high-performance key-value store. > It now have ACLs and SASL, but do not related to ranger. > We suggest to add support to it. > > Pegasus have done its work. > https://github.com/apache/incubator-pegasus/issues/1054 > Could anyone do some work to merge this definition into ranger? > > > Diffs > ----- > > > agents-common/src/main/java/org/apache/ranger/plugin/store/EmbeddedServiceDefsUtil.java > 92ce183bb > > agents-common/src/main/resources/service-defs/ranger-servicedef-pegasus.json > PRE-CREATION > distro/src/main/assembly/admin-web.xml 245d9ca09 > plugin-pegasus/pom.xml PRE-CREATION > > plugin-pegasus/src/main/java/org/apache/ranger/services/pegasus/RangerServicePegasus.java > PRE-CREATION > pom.xml cec2390f2 > > > Diff: https://reviews.apache.org/r/74942/diff/1/ > > > Testing > ------- > > Done by our mini site. > > > Thanks, > > Kirby Zhou > >
