[ https://issues.apache.org/jira/browse/YARN-5656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15497412#comment-15497412 ]
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 (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org