> On June 8, 2017, 10:05 a.m., Colm O hEigeartaigh wrote: > > "Currently it only support solr" - isn't hdfs supported as well here? > > Qiang Zhang wrote: > The Ranger Admin uses the configuration item to determine log should be > queried from db or solr. It only supported query logs from db or solr using > Ranger Admin UI. Isn't hdfs supported as well here. > > Qiang Zhang wrote: > The hdfs isn't supported here. > > Colm O hEigeartaigh wrote: > OK thanks for the clarification. Maybe HDFS auditing is only supported > for the plugins? In any case, please change "Currently it only support solr." > to "Currently only solr is supported.".
Ok. I had fixed it. - Qiang ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59908/#review177291 ----------------------------------------------------------- On June 8, 2017, 7:05 a.m., Qiang Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59908/ > ----------------------------------------------------------- > > (Updated June 8, 2017, 7:05 a.m.) > > > Review request for ranger, Alok Lal, Ankita Sinha, Don Bosco Durai, Colm O > hEigeartaigh, Gautam Borad, Madhan Neethiraj, Ramesh Mani, Selvamohan > Neethiraj, and Velmurugan Periasamy. > > > Bugs: RANGER-1604 > https://issues.apache.org/jira/browse/RANGER-1604 > > > Repository: ranger > > > Description > ------- > > DB based auditing had been removed in RANGER-900. But some configurations and > codes are not removed accordingly in Ranger Admin. It often makes the user > feel confused such as RANGER-1286. We should do the following modification: > 1. Add description information to configuration file to remind users don't > use db to store audit log.. > 2. Remove invalid code from setup.sh to eliminate misunderstanding. In order > to avoid the introduction of the new issue, I do not modify db_setup.py and > dba_script.py. > > I carefully tested the issue. > > > Diffs > ----- > > security-admin/scripts/install.properties 06a1b89 > security-admin/scripts/setup.sh ef61e8d > > > Diff: https://reviews.apache.org/r/59908/diff/1/ > > > Testing > ------- > > > Thanks, > > Qiang Zhang > >