----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70057/#review213730 -----------------------------------------------------------
embeddedwebserver/src/main/java/org/apache/ranger/server/tomcat/SolrSetupUtil.java Lines 63 (patched) <https://reviews.apache.org/r/70057/#comment299706> overall, it's much closer for that how i thought it first how it should work. few things: - would make sense to rename this thread to something else, because it's not realy a util class, maybe SolrCollectionBoostrapper or something similar. - not only ranger should be used as sasl user (like ranger should be able to edit the configs, but solr user as well), it should be a way to provide additional users, because with Ambari/CM, that will be needed. - what if solrconfig.xml is changed on the filesystem? because it should be uploaded again by the bootstrapper on startup. (TTL can be changed in the solrconfig.xml, but in that case we can just upload that file, as for example, managed-schema mostly edited by Solr itself, so that one should not be touched) ... so that solrconfig.xml file should be downloaded to the application memory from the znode and check is it differs from the uploaded one. - an atomic shared variable would be useful (for example that can be passed by the constructor) that can be used even on REST API side to filter out any Solr Query and answer that Solr is not ready yet (or ZK) ... but this is not really essential of course - Oliver Szabo On March 14, 2019, 3:31 p.m., bhavik patel wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70057/ > ----------------------------------------------------------- > > (Updated March 14, 2019, 3:31 p.m.) > > > Review request for ranger, Ankita Sinha, Don Bosco Durai, Gautam Borad, Abhay > Kulkarni, Madhan Neethiraj, Oliver Szabo, Pradeep Agrawal, Ramesh Mani, > Selvamohan Neethiraj, Sailaja Polavarapu, and Velmurugan Periasamy. > > > Bugs: RANGER-2324 > https://issues.apache.org/jira/browse/RANGER-2324 > > > Repository: ranger > > > Description > ------- > > We are handling the solr bootstrapping in below mentioned manner for Ranger > 1.) Connection to solr > 2.) Upload Configuration > 3.) Create Collection > 4.) Setting ACL > > > Diffs > ----- > > > embeddedwebserver/src/main/java/org/apache/ranger/server/tomcat/EmbeddedServer.java > 8d32352 > > embeddedwebserver/src/main/java/org/apache/ranger/server/tomcat/SolrSetupUtil.java > PRE-CREATION > security-admin/scripts/install.properties fdcee1b > security-admin/scripts/setup.sh bd4bd4c > security-admin/src/main/resources/conf.dist/ranger-admin-default-site.xml > 686f720 > security-admin/src/main/resources/conf.dist/ranger-admin-site.xml 4d4a1de > src/main/assembly/admin-web.xml 8ea728b > > > Diff: https://reviews.apache.org/r/70057/diff/3/ > > > Testing > ------- > > Tested Below Scenario on ranger manual start / restart > 1.) Solr configuration were uploaded successfully > 2.) Solr collections were created successfully > 3.) ACL were setup as required. > > > Thanks, > > bhavik patel > >
