Sean Po commented on YARN-5656:

I think it is safe to remove the check entirely given the two scenarios:

1. If ACLS are enabled, then a check that ensures that the reservation is being 
updated by an admin / by the user who submitted the reservation is redundant. 

2. Otherwise, if ACLs are not enabled, we are saying that anyone can update any 
reservation, so the check would be invalid.

> ReservationACLsTestBase fails on trunk
> --------------------------------------
>                 Key: YARN-5656
>                 URL: https://issues.apache.org/jira/browse/YARN-5656
>             Project: Hadoop YARN
>          Issue Type: Test
>            Reporter: Sean Po
>            Assignee: Sean Po
>         Attachments: YARN-5656.v1.patch
> ReservationACLsTestBase fails when verifying that a reservation can be 
> successfully updated by a user who did not submit the reservation who also 
> has an admin ACL.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to