You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@falcon.apache.org by Raghav Gautam <ra...@apache.org> on 2014/10/16 19:00:45 UTC
Review Request 26812: all the entities must have acl
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26812/
-----------------------------------------------------------
Review request for Falcon.
Bugs: falcon-812
https://issues.apache.org/jira/browse/falcon-812
Repository: falcon-git
Description
-------
This fix adds current user and his group to all the entities.
Diffs
-----
falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/BundleUtil.java 3b2f8dd
Diff: https://reviews.apache.org/r/26812/diff/
Testing
-------
Ran EmbeddedPigScriptTest with and without authorization enabled.
Entity submission are going through in both cases.
Thanks,
Raghav Gautam
Re: Review Request 26812: all the entities must have acl
Posted by Arpit Gupta <ar...@hortonworks.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26812/#review57233
-----------------------------------------------------------
falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/BundleUtil.java
<https://reviews.apache.org/r/26812/#comment97776>
user should be set to current user by default.
- Arpit Gupta
On Oct. 16, 2014, 5 p.m., Raghav Gautam wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/26812/
> -----------------------------------------------------------
>
> (Updated Oct. 16, 2014, 5 p.m.)
>
>
> Review request for Falcon.
>
>
> Bugs: falcon-812
> https://issues.apache.org/jira/browse/falcon-812
>
>
> Repository: falcon-git
>
>
> Description
> -------
>
> This fix adds current user and his group to all the entities.
>
>
> Diffs
> -----
>
> falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/BundleUtil.java 3b2f8dd
>
> Diff: https://reviews.apache.org/r/26812/diff/
>
>
> Testing
> -------
>
> Ran EmbeddedPigScriptTest with and without authorization enabled.
> Entity submission are going through in both cases.
>
>
> Thanks,
>
> Raghav Gautam
>
>
Re: Review Request 26812: all the entities must have acl
Posted by Raghav Gautam <ra...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26812/
-----------------------------------------------------------
(Updated Oct. 17, 2014, 4:04 p.m.)
Review request for Falcon.
Bugs: falcon-812
https://issues.apache.org/jira/browse/falcon-812
Repository: falcon-git
Description
-------
This fix adds current user and his group to all the entities.
Diffs (updated)
-----
falcon-regression/merlin-core/src/main/java/org/apache/falcon/regression/core/util/BundleUtil.java 3b2f8dd
Diff: https://reviews.apache.org/r/26812/diff/
Testing
-------
Ran EmbeddedPigScriptTest with and without authorization enabled.
Entity submission are going through in both cases.
Thanks,
Raghav Gautam