Re: Review Request 54966: Fix typo in druid service postgresql

2016-12-22 Thread Slim Bouguerra
ons_withMysql (test_stack_advisor.TestHDP26StackAdvisor) ... ok -- Ran 6 tests in 0.200s OK Thanks, Slim Bouguerra

Re: Review Request 54979: Typo in the PostgreSQL database name for druid service

2016-12-22 Thread Slim Bouguerra
> On Dec. 22, 2016, 2:55 p.m., Slim Bouguerra wrote: > > also i have sent the patch to Apache JIRA > > https://issues.apache.org/jira/browse/AMBARI-19278 but it is not applied. > > Attila Magyar wrote: > I think it was not applied because it was created from the am

Re: Review Request 54979: Typo in the PostgreSQL database name for druid service

2016-12-22 Thread Slim Bouguerra
> On Dec. 22, 2016, 2:55 p.m., Slim Bouguerra wrote: > > also i have sent the patch to Apache JIRA > > https://issues.apache.org/jira/browse/AMBARI-19278 but it is not applied. > > Attila Magyar wrote: > I think it was not applied because it was created from the am

Re: Review Request 54979: Typo in the PostgreSQL database name for druid service

2016-12-22 Thread Slim Bouguerra
://issues.apache.org/jira/browse/AMBARI-19278 but it is not applied. - Slim Bouguerra On Dec. 22, 2016, 2:51 p.m., Attila Magyar wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 54979: Typo in the PostgreSQL database name for druid service

2016-12-22 Thread Slim Bouguerra
/ can you please review it ? - Slim Bouguerra On Dec. 22, 2016, 2:51 p.m., Attila Magyar wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 50803: [Ambari-17981] Integrate Druid With Ambari

2016-10-24 Thread Slim Bouguerra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50803/#review153707 --- Ship it! Ship It! - Slim Bouguerra On Oct. 22, 2016, 1:21

Re: Review Request 50803: [Ambari-19781] Integrate Druid With Ambari

2016-10-14 Thread Slim Bouguerra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50803/#review152683 --- Ship it! Ship It! - Slim Bouguerra On Oct. 14, 2016, 11:15

Re: Review Request 50803: [Ambari-19781] Integrate Druid With Ambari

2016-10-11 Thread Slim Bouguerra
ent220971> this is using curl which means it will fail if curl is not installed or part of the bin path. Not an issue but it will be better to use python http clients. - Slim Bouguerra On Oct. 11, 2016, 2:44 p.m., Nishant Ba