You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ambari.apache.org by Nate Cole <nc...@hortonworks.com> on 2015/09/25 21:47:33 UTC
Review Request 38771: RU cluster in hung state while trying to perform
downgrade
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38771/
-----------------------------------------------------------
Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Jayush Luniya.
Bugs: AMBARI-13245
https://issues.apache.org/jira/browse/AMBARI-13245
Repository: ambari
Description
-------
The recently added {{Parallel}} class has a default thread timeout that is too low. As a result, EclipseLink code is deadlocking and preventing queries from executing.
Diffs
-----
ambari-server/src/main/java/org/apache/ambari/server/utils/Parallel.java a67ee5c
Diff: https://reviews.apache.org/r/38771/diff/
Testing
-------
Manual testing only. No automated tests due to constructor changes only.
Thanks,
Nate Cole
Re: Review Request 38771: RU cluster in hung state while trying to
perform downgrade
Posted by Jayush Luniya <jl...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38771/#review100673
-----------------------------------------------------------
Ship it!
Ship It!
- Jayush Luniya
On Sept. 25, 2015, 7:47 p.m., Nate Cole wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38771/
> -----------------------------------------------------------
>
> (Updated Sept. 25, 2015, 7:47 p.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Jayush Luniya.
>
>
> Bugs: AMBARI-13245
> https://issues.apache.org/jira/browse/AMBARI-13245
>
>
> Repository: ambari
>
>
> Description
> -------
>
> The recently added {{Parallel}} class has a default thread timeout that is too low. As a result, EclipseLink code is deadlocking and preventing queries from executing.
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/utils/Parallel.java a67ee5c
>
> Diff: https://reviews.apache.org/r/38771/diff/
>
>
> Testing
> -------
>
> Manual testing only. No automated tests due to constructor changes only.
>
>
> Thanks,
>
> Nate Cole
>
>
Re: Review Request 38771: RU cluster in hung state while trying to
perform downgrade
Posted by Jonathan Hurley <jh...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38771/#review100669
-----------------------------------------------------------
Ship it!
ambari-server/src/main/java/org/apache/ambari/server/utils/Parallel.java (line 50)
<https://reviews.apache.org/r/38771/#comment157930>
On a single core system, could 8 be too much?
ambari-server/src/main/java/org/apache/ambari/server/utils/Parallel.java (line 53)
<https://reviews.apache.org/r/38771/#comment157929>
Should probably !!! exactly why we changed this value.
- Jonathan Hurley
On Sept. 25, 2015, 3:47 p.m., Nate Cole wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38771/
> -----------------------------------------------------------
>
> (Updated Sept. 25, 2015, 3:47 p.m.)
>
>
> Review request for Ambari, Alejandro Fernandez, Jonathan Hurley, and Jayush Luniya.
>
>
> Bugs: AMBARI-13245
> https://issues.apache.org/jira/browse/AMBARI-13245
>
>
> Repository: ambari
>
>
> Description
> -------
>
> The recently added {{Parallel}} class has a default thread timeout that is too low. As a result, EclipseLink code is deadlocking and preventing queries from executing.
>
>
> Diffs
> -----
>
> ambari-server/src/main/java/org/apache/ambari/server/utils/Parallel.java a67ee5c
>
> Diff: https://reviews.apache.org/r/38771/diff/
>
>
> Testing
> -------
>
> Manual testing only. No automated tests due to constructor changes only.
>
>
> Thanks,
>
> Nate Cole
>
>