Re: Review Request 47377: Fix for parsing multiple lines in output while looking for created 'llap package' name.

2016-05-13 Thread Swapan Shridhar
> On May 14, 2016, 12:19 a.m., Sumit Mohanty wrote: > > ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/package/scripts/hive_server_interactive.py, > > line 240 > > > > > > We should add an explicit

Re: Review Request 47377: Fix for parsing multiple lines in output while looking for created 'llap package' name.

2016-05-13 Thread Swapan Shridhar
> On May 14, 2016, 12:20 a.m., Sumit Mohanty wrote: > > You can add the test later but can you add the JIRA ID to the review > > request? > > Sumit Mohanty wrote: > +1. Did not realize that JIRA is inaccessible. Once you have a JIRA > update the review and commit it. Updated JIRA. - Swa

Re: Review Request 47377: Fix for parsing multiple lines in output while looking for created 'llap package' name.

2016-05-13 Thread Sumit Mohanty
> On May 14, 2016, 12:20 a.m., Sumit Mohanty wrote: > > You can add the test later but can you add the JIRA ID to the review > > request? +1. Did not realize that JIRA is inaccessible. Once you have a JIRA update the review and commit it. - Sumit ---

Re: Review Request 47377: Fix for parsing multiple lines in output while looking for created 'llap package' name.

2016-05-13 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47377/#review133235 --- You can add the test later but can you add the JIRA ID to the revi

Re: Review Request 47377: Fix for parsing multiple lines in output while looking for created 'llap package' name.

2016-05-13 Thread Sumit Mohanty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47377/#review133234 --- ambari-server/src/main/resources/common-services/HIVE/0.12.0.2.0/

Review Request 47377: Fix for parsing multiple lines in output while looking for created 'llap package' name.

2016-05-13 Thread Swapan Shridhar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47377/ --- Review request for Ambari, Jaimin Jetly and Sumit Mohanty. Repository: ambari