Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-19 Thread Sandor Magyari
to:reviews@ambari.apache.org>> Subject: Re: Review Request 51843: rolling restart datanode cluster name in audit log show null This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/ On 九月 16th, 2016, 4:35 p.m. UTC, Attila Doroszlai wrote: amba

Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-18 Thread wang yaoxin
> On 九月 16, 2016, 4:35 p.m., Attila Doroszlai wrote: > > ambari-server/src/test/java/org/apache/ambari/server/audit/request/creator/RequestEventCreatorTest.java, > > line 72 > > > > > > After adding the missing impor

Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-18 Thread wang yaoxin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/ --- (Updated 九月 18, 2016, 9:20 a.m.) Review request for Ambari, Jonathan Hurley, Ol

Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-16 Thread Attila Doroszlai
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/#review149209 --- ambari-server/src/test/java/org/apache/ambari/server/audit/reques

Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-16 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/#review149199 --- Ship it! Ship It! - Robert Nettleton On Sept. 16, 2016, 1:44

Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-16 Thread wang yaoxin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/ --- (Updated 九月 16, 2016, 1:44 p.m.) Review request for Ambari, Jonathan Hurley, Ol

Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-16 Thread Robert Nettleton
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/#review149198 --- ambari-server/src/test/java/org/apache/ambari/server/audit/reques

Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-16 Thread Sandor Magyari
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/#review149197 --- Ship it! Could you please paste the unitest result to 'Testing

Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-16 Thread wang yaoxin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/ --- (Updated 九月 16, 2016, 12:54 p.m.) Review request for Ambari, Jonathan Hurley, O

Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-16 Thread wang yaoxin
> On 九月 16, 2016, 10:58 a.m., Oliver Szabo wrote: > > after applying the diff i got: > > [ERROR] > > /Users/oszabo/prj/ambari/ambari-server/src/main/java/org/apache/ambari/server/audit/request/eventcreator/RequestEventCreator.java:[103,5] > > cannot find symbol > > symbol: class Map > > l

Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-16 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/#review149186 --- after applying the diff i got: [ERROR] /Users/oszabo/prj/ambari/a

Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-16 Thread wang yaoxin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/ --- (Updated 九月 16, 2016, 10:06 a.m.) Review request for Ambari, Jonathan Hurley, O

Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-14 Thread Oliver Szabo
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/#review148894 --- Ship it! Ship It! - Oliver Szabo On Sept. 14, 2016, 8:31 a.m

Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-14 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/#review148872 --- Ship it! Ship It! - Sebastian Toader On Sept. 14, 2016, 8:04

Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-13 Thread wang yaoxin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/ --- (Updated 九月 14, 2016, 6:04 a.m.) Review request for Ambari, Jonathan Hurley, Ro

Re: Review Request 51843: rolling restart datanode cluster name in audit log show null

2016-09-13 Thread Sebastian Toader
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51843/#review148703 --- ambari-server/src/main/java/org/apache/ambari/server/audit/reques