Re: Review Request 62376: The JAVA_HOME property is invalid in install.properties for Solr for Ranger Audits. The solr started fail after installed and configured Solr for Ranger Audits.

2017-09-20 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62376/#review185795 --- Ship it! Ship It! - Colm O hEigeartaigh On Sept. 19, 2017, 1

Re: Review Request 62376: The JAVA_HOME property is invalid in install.properties for Solr for Ranger Audits. The solr started fail after installed and configured Solr for Ranger Audits.

2017-09-19 Thread pengjianhua
> On 九月 19, 2017, 11:43 a.m., Colm O hEigeartaigh wrote: > > I wonder if we need to support JAVA_HOME at all here in the solr audit > > install.properties? Why not just take it from the environment like all of > > the other plugins? > > pengjianhua wrote: > The case will be very passive wh

Re: Review Request 62376: The JAVA_HOME property is invalid in install.properties for Solr for Ranger Audits. The solr started fail after installed and configured Solr for Ranger Audits.

2017-09-19 Thread Colm O hEigeartaigh
> On Sept. 19, 2017, 11:43 a.m., Colm O hEigeartaigh wrote: > > I wonder if we need to support JAVA_HOME at all here in the solr audit > > install.properties? Why not just take it from the environment like all of > > the other plugins? > > pengjianhua wrote: > The case will be very passive

Re: Review Request 62376: The JAVA_HOME property is invalid in install.properties for Solr for Ranger Audits. The solr started fail after installed and configured Solr for Ranger Audits.

2017-09-19 Thread pengjianhua
> On 九月 19, 2017, 11:43 a.m., Colm O hEigeartaigh wrote: > > I wonder if we need to support JAVA_HOME at all here in the solr audit > > install.properties? Why not just take it from the environment like all of > > the other plugins? > > pengjianhua wrote: > The case will be very passive wh

Re: Review Request 62376: The JAVA_HOME property is invalid in install.properties for Solr for Ranger Audits. The solr started fail after installed and configured Solr for Ranger Audits.

2017-09-19 Thread pengjianhua
> On 九月 19, 2017, 11:43 a.m., Colm O hEigeartaigh wrote: > > I wonder if we need to support JAVA_HOME at all here in the solr audit > > install.properties? Why not just take it from the environment like all of > > the other plugins? > > pengjianhua wrote: > The case will be very passive wh

Re: Review Request 62376: The JAVA_HOME property is invalid in install.properties for Solr for Ranger Audits. The solr started fail after installed and configured Solr for Ranger Audits.

2017-09-19 Thread Colm O hEigeartaigh
> On Sept. 19, 2017, 11:43 a.m., Colm O hEigeartaigh wrote: > > I wonder if we need to support JAVA_HOME at all here in the solr audit > > install.properties? Why not just take it from the environment like all of > > the other plugins? > > pengjianhua wrote: > The case will be very passive

Re: Review Request 62376: The JAVA_HOME property is invalid in install.properties for Solr for Ranger Audits. The solr started fail after installed and configured Solr for Ranger Audits.

2017-09-19 Thread pengjianhua
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62376/ --- (Updated 九月 19, 2017, 12:45 p.m.) Review request for ranger, Alok Lal, Ankita S

Re: Review Request 62376: The JAVA_HOME property is invalid in install.properties for Solr for Ranger Audits. The solr started fail after installed and configured Solr for Ranger Audits.

2017-09-19 Thread pengjianhua
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62376/ --- (Updated 九月 19, 2017, 12:39 p.m.) Review request for ranger, Alok Lal, Ankita S