You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Alok Lal <al...@hortonworks.com> on 2015/09/08 21:36:00 UTC
Review Request 38189: RANGER-639 Package commons-lang,
which is a required dependency, as part of storm plugin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38189/
-----------------------------------------------------------
Review request for ranger and Selvamohan Neethiraj.
Bugs: RANGER-639
https://issues.apache.org/jira/browse/RANGER-639
Repository: ranger
Description
-------
Package commons-lang, which is a required dependency, as part of storm plugin. Currently this was part of storm distro. However, if and when storm starts to shade it, ranger plugin would run into problems.. So we must include it as part of ranger tar-gz.
Diffs
-----
src/main/assembly/storm-agent.xml aacdb78
Diff: https://reviews.apache.org/r/38189/diff/
Testing
-------
current tests work.
Thanks,
Alok Lal
Re: Review Request 38189: RANGER-639 Package commons-lang,
which is a required dependency, as part of storm plugin
Posted by Selvamohan Neethiraj <sn...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38189/#review98060
-----------------------------------------------------------
Ship it!
Ship It!
- Selvamohan Neethiraj
On Sept. 8, 2015, 3:36 p.m., Alok Lal wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38189/
> -----------------------------------------------------------
>
> (Updated Sept. 8, 2015, 3:36 p.m.)
>
>
> Review request for ranger and Selvamohan Neethiraj.
>
>
> Bugs: RANGER-639
> https://issues.apache.org/jira/browse/RANGER-639
>
>
> Repository: ranger
>
>
> Description
> -------
>
> Package commons-lang, which is a required dependency, as part of storm plugin. Currently this was part of storm distro. However, if and when storm starts to shade it, ranger plugin would run into problems.. So we must include it as part of ranger tar-gz.
>
>
> Diffs
> -----
>
> src/main/assembly/storm-agent.xml aacdb78
>
> Diff: https://reviews.apache.org/r/38189/diff/
>
>
> Testing
> -------
>
> current tests work.
>
>
> Thanks,
>
> Alok Lal
>
>