You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@ambari.apache.org by Andrew Onischuk <ao...@hortonworks.com> on 2016/03/22 15:57:41 UTC
Re: Review Request 43049: Oozie should update war after adding Falcon
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43049/
-----------------------------------------------------------
(Updated March 22, 2016, 2:57 p.m.)
Review request for Ambari and Dmitro Lisnichenko.
Bugs: AMBARI-14863
https://issues.apache.org/jira/browse/AMBARI-14863
Repository: ambari
Description
-------
After adding Falcon, falcon-oozie-el-extension-*.jar is added to oozie-
server/libext
Oozie war should be updated.
Diffs (updated)
-----
ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py c503a38
ambari-server/src/main/resources/scripts/Ambaripreupload.py 941539c
ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py 407bb2f
Diff: https://reviews.apache.org/r/43049/diff/
Testing
-------
mvn clean test
Thanks,
Andrew Onischuk
Re: Review Request 43049: Oozie should update war after adding Falcon
Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43049/#review125070
-----------------------------------------------------------
Ship it!
Ship It!
- Alejandro Fernandez
On March 23, 2016, 4:04 p.m., Andrew Onischuk wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43049/
> -----------------------------------------------------------
>
> (Updated March 23, 2016, 4:04 p.m.)
>
>
> Review request for Ambari and Dmitro Lisnichenko.
>
>
> Bugs: AMBARI-14863
> https://issues.apache.org/jira/browse/AMBARI-14863
>
>
> Repository: ambari
>
>
> Description
> -------
>
> After adding Falcon, falcon-oozie-el-extension-*.jar is added to oozie-
> server/libext
> Oozie war should be updated.
>
>
> Diffs
> -----
>
> ambari-common/src/main/python/resource_management/libraries/functions/oozie_prepare_war.py PRE-CREATION
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py 5587380
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_server_upgrade.py d26b89d
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/params_linux.py f92d90c
> ambari-server/src/main/resources/scripts/Ambaripreupload.py cbec3cf
> ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py ba61b3d
>
> Diff: https://reviews.apache.org/r/43049/diff/
>
>
> Testing
> -------
>
> mvn clean test
>
>
> Thanks,
>
> Andrew Onischuk
>
>
Re: Review Request 43049: Oozie should update war after adding Falcon
Posted by Dmitro Lisnichenko <dl...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43049/#review125051
-----------------------------------------------------------
Ship it!
Ship It!
- Dmitro Lisnichenko
On March 23, 2016, 6:04 p.m., Andrew Onischuk wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43049/
> -----------------------------------------------------------
>
> (Updated March 23, 2016, 6:04 p.m.)
>
>
> Review request for Ambari and Dmitro Lisnichenko.
>
>
> Bugs: AMBARI-14863
> https://issues.apache.org/jira/browse/AMBARI-14863
>
>
> Repository: ambari
>
>
> Description
> -------
>
> After adding Falcon, falcon-oozie-el-extension-*.jar is added to oozie-
> server/libext
> Oozie war should be updated.
>
>
> Diffs
> -----
>
> ambari-common/src/main/python/resource_management/libraries/functions/oozie_prepare_war.py PRE-CREATION
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py 5587380
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_server_upgrade.py d26b89d
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/params_linux.py f92d90c
> ambari-server/src/main/resources/scripts/Ambaripreupload.py cbec3cf
> ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py ba61b3d
>
> Diff: https://reviews.apache.org/r/43049/diff/
>
>
> Testing
> -------
>
> mvn clean test
>
>
> Thanks,
>
> Andrew Onischuk
>
>
Re: Review Request 43049: Oozie should update war after adding Falcon
Posted by Andrew Onischuk <ao...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43049/
-----------------------------------------------------------
(Updated March 23, 2016, 4:04 p.m.)
Review request for Ambari and Dmitro Lisnichenko.
Bugs: AMBARI-14863
https://issues.apache.org/jira/browse/AMBARI-14863
Repository: ambari
Description
-------
After adding Falcon, falcon-oozie-el-extension-*.jar is added to oozie-
server/libext
Oozie war should be updated.
Diffs (updated)
-----
ambari-common/src/main/python/resource_management/libraries/functions/oozie_prepare_war.py PRE-CREATION
ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py 5587380
ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie_server_upgrade.py d26b89d
ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/params_linux.py f92d90c
ambari-server/src/main/resources/scripts/Ambaripreupload.py cbec3cf
ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py ba61b3d
Diff: https://reviews.apache.org/r/43049/diff/
Testing
-------
mvn clean test
Thanks,
Andrew Onischuk
Re: Review Request 43049: Oozie should update war after adding Falcon
Posted by Dmitro Lisnichenko <dl...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43049/#review124803
-----------------------------------------------------------
Ship it!
Ship It!
- Dmitro Lisnichenko
On March 22, 2016, 4:57 p.m., Andrew Onischuk wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43049/
> -----------------------------------------------------------
>
> (Updated March 22, 2016, 4:57 p.m.)
>
>
> Review request for Ambari and Dmitro Lisnichenko.
>
>
> Bugs: AMBARI-14863
> https://issues.apache.org/jira/browse/AMBARI-14863
>
>
> Repository: ambari
>
>
> Description
> -------
>
> After adding Falcon, falcon-oozie-el-extension-*.jar is added to oozie-
> server/libext
> Oozie war should be updated.
>
>
> Diffs
> -----
>
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py c503a38
> ambari-server/src/main/resources/scripts/Ambaripreupload.py 941539c
> ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py 407bb2f
>
> Diff: https://reviews.apache.org/r/43049/diff/
>
>
> Testing
> -------
>
> mvn clean test
>
>
> Thanks,
>
> Andrew Onischuk
>
>
Re: Review Request 43049: Oozie should update war after adding Falcon
Posted by Alejandro Fernandez <af...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/43049/#review124829
-----------------------------------------------------------
ambari-server/src/main/resources/scripts/Ambaripreupload.py (line 338)
<https://reviews.apache.org/r/43049/#comment187524>
This should be in a function to avoid duplicating code.
Or at a minimum, include a comment in both places indicating the other location to change.
- Alejandro Fernandez
On March 22, 2016, 2:57 p.m., Andrew Onischuk wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/43049/
> -----------------------------------------------------------
>
> (Updated March 22, 2016, 2:57 p.m.)
>
>
> Review request for Ambari and Dmitro Lisnichenko.
>
>
> Bugs: AMBARI-14863
> https://issues.apache.org/jira/browse/AMBARI-14863
>
>
> Repository: ambari
>
>
> Description
> -------
>
> After adding Falcon, falcon-oozie-el-extension-*.jar is added to oozie-
> server/libext
> Oozie war should be updated.
>
>
> Diffs
> -----
>
> ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/package/scripts/oozie.py c503a38
> ambari-server/src/main/resources/scripts/Ambaripreupload.py 941539c
> ambari-server/src/test/python/stacks/2.0.6/OOZIE/test_oozie_server.py 407bb2f
>
> Diff: https://reviews.apache.org/r/43049/diff/
>
>
> Testing
> -------
>
> mvn clean test
>
>
> Thanks,
>
> Andrew Onischuk
>
>