You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@falcon.apache.org by "Ajay Yadav (JIRA)" <ji...@apache.org> on 2014/12/23 03:50:13 UTC
[jira] [Comment Edited] (FALCON-967) Better Error message while
feed submission with wrong perms
[ https://issues.apache.org/jira/browse/FALCON-967?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14256507#comment-14256507 ]
Ajay Yadav edited comment on FALCON-967 at 12/23/14 2:49 AM:
-------------------------------------------------------------
I think it's slightly confusing as of now and suggested change will be more helpful.
was (Author: ajayyadava):
I think it's slightly confusing and suggested message will be more helpful.
> Better Error message while feed submission with wrong perms
> -----------------------------------------------------------
>
> Key: FALCON-967
> URL: https://issues.apache.org/jira/browse/FALCON-967
> Project: Falcon
> Issue Type: Bug
> Reporter: karan kumar
> Priority: Minor
>
> Suppose a feed at path <a/b/c/yyyy/mm/dd/e> is submitted with acl user as "user1".
> On hdfs path <a/b/c> has the fileowner as "user2".
> Falcon fails to submit the feed with this error message.
> Caused by: org.apache.falcon.FalconException: Feed ACL owner user1 doesn't match the actual file owner user2 for location /a/b/${YEAR}-${MONTH}-${DAY}/data
>
> In the case where /a/b/${YEAR}-${MONTH}-${DAY}/data has perms with user1 falcon only checks for perms till /a/b
> {noformat}
> private String getRelativePath(Location location) {
> // if the path contains variables, locate on the "parent" path (just before first variable usage)
> Matcher matcher = FeedDataPath.PATTERN.matcher(location.getPath());
> boolean timedPath = matcher.find();
> if (timedPath) {
> return location.getPath().substring(0, matcher.start());
> }
> return location.getPath();
> }
> {noformat}
> but the error message prints the full feed path.
> The error message should contain path till /a/b IMHO.
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)