You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@falcon.apache.org by "karan kumar (JIRA)" <ji...@apache.org> on 2014/12/22 16:27:14 UTC

[jira] [Commented] (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=14255830#comment-14255830 ] 

karan kumar commented on FALCON-967:
------------------------------------

 If the community thinks its a valid bug would like to work on it.

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