falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ajay Yadav (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (FALCON-967) Better Error message while feed submission with wrong perms
Date Tue, 23 Dec 2014 02:50:13 GMT

    [ 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)

Mime
View raw message