Re: Review Request 46000: Parallel execution should only be allowed on commands that have auto retry enabled

2016-04-11 Thread Mahadev Konar 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46000/#review128280
---


Ship it!




Ship It!

- Mahadev Konar


On April 11, 2016, 4:27 a.m., Sumit Mohanty wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46000/
> ---
> 
> (Updated April 11, 2016, 4:27 a.m.)
> 
> 
> Review request for Ambari, Sandor Magyari and Sebastian Toader.
> 
> 
> Bugs: AMBARI-15795
> https://issues.apache.org/jira/browse/AMBARI-15795
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Parallel execution should only be allowed on commands that have auto retry 
> enabled
> 
> 
> Diffs
> -
> 
>   ambari-agent/src/main/python/ambari_agent/ActionQueue.py 4d4103e 
>   ambari-agent/src/test/python/ambari_agent/TestActionQueue.py 466c3a8 
> 
> Diff: https://reviews.apache.org/r/46000/diff/
> 
> 
> Testing
> ---
> 
> mvn test against ambari-agent passed. New unit tests added.
> 
> 
> Thanks,
> 
> Sumit Mohanty
> 
>

Re: Review Request 46000: Parallel execution should only be allowed on commands that have auto retry enabled

2016-04-11 Thread Sumit Mohanty 


> On April 11, 2016, 6:32 p.m., Alejandro Fernandez wrote:
> > ambari-agent/src/main/python/ambari_agent/ActionQueue.py, line 163
> > 
> >
> > What was failing because of these multiple threads?

Parallel start/stop of components.


- Sumit


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46000/#review128196
---


On April 11, 2016, 4:27 a.m., Sumit Mohanty wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46000/
> ---
> 
> (Updated April 11, 2016, 4:27 a.m.)
> 
> 
> Review request for Ambari, Sandor Magyari and Sebastian Toader.
> 
> 
> Bugs: AMBARI-15795
> https://issues.apache.org/jira/browse/AMBARI-15795
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Parallel execution should only be allowed on commands that have auto retry 
> enabled
> 
> 
> Diffs
> -
> 
>   ambari-agent/src/main/python/ambari_agent/ActionQueue.py 4d4103e 
>   ambari-agent/src/test/python/ambari_agent/TestActionQueue.py 466c3a8 
> 
> Diff: https://reviews.apache.org/r/46000/diff/
> 
> 
> Testing
> ---
> 
> mvn test against ambari-agent passed. New unit tests added.
> 
> 
> Thanks,
> 
> Sumit Mohanty
> 
>

Re: Review Request 46000: Parallel execution should only be allowed on commands that have auto retry enabled

2016-04-11 Thread Sandor Magyari 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46000/#review128086
---


Ship it!




Ship It!

- Sandor Magyari


On April 11, 2016, 4:27 a.m., Sumit Mohanty wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46000/
> ---
> 
> (Updated April 11, 2016, 4:27 a.m.)
> 
> 
> Review request for Ambari, Sandor Magyari and Sebastian Toader.
> 
> 
> Bugs: AMBARI-15795
> https://issues.apache.org/jira/browse/AMBARI-15795
> 
> 
> Repository: ambari
> 
> 
> Description
> ---
> 
> Parallel execution should only be allowed on commands that have auto retry 
> enabled
> 
> 
> Diffs
> -
> 
>   ambari-agent/src/main/python/ambari_agent/ActionQueue.py 4d4103e 
>   ambari-agent/src/test/python/ambari_agent/TestActionQueue.py 466c3a8 
> 
> Diff: https://reviews.apache.org/r/46000/diff/
> 
> 
> Testing
> ---
> 
> mvn test against ambari-agent passed. New unit tests added.
> 
> 
> Thanks,
> 
> Sumit Mohanty
> 
>

Review Request 46000: Parallel execution should only be allowed on commands that have auto retry enabled

2016-04-10 Thread Sumit Mohanty 

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/46000/
---

Review request for Ambari, Sandor Magyari and Sebastian Toader.


Bugs: AMBARI-15795
https://issues.apache.org/jira/browse/AMBARI-15795


Repository: ambari


Description
---

Parallel execution should only be allowed on commands that have auto retry 
enabled


Diffs
-

  ambari-agent/src/main/python/ambari_agent/ActionQueue.py 4d4103e 
  ambari-agent/src/test/python/ambari_agent/TestActionQueue.py 466c3a8 

Diff: https://reviews.apache.org/r/46000/diff/


Testing
---

mvn test against ambari-agent passed. New unit tests added.


Thanks,

Sumit Mohanty