[ https://issues.apache.org/jira/browse/YARN-4420?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15138277#comment-15138277 ]
Subru Krishnan commented on YARN-4420: -------------------------------------- Thanks [~seanpo03] for addressing all my comments. The latest patch (v5) LGTM, +1. A minor nit, fields in the DAO like _resourceAllocations_ and _reservations_ should be of base class List type. [~leftnoteasy] or [~curino], can you guys take a quick look. > Add REST API for List Reservations > ---------------------------------- > > Key: YARN-4420 > URL: https://issues.apache.org/jira/browse/YARN-4420 > Project: Hadoop YARN > Issue Type: Sub-task > Components: capacity scheduler, fairscheduler, resourcemanager > Reporter: Sean Po > Assignee: Sean Po > Priority: Minor > Attachments: YARN-4420.v1.patch, YARN-4420.v2.patch, > YARN-4420.v3.patch, YARN-4420.v4.patch, YARN-4420.v5.patch > > > This JIRA tracks changes to the REST APIs of the reservation system and > enables querying the reservation on which reservations exists by "time-range, > and reservation-id". > This task has a dependency on YARN-4340. -- This message was sent by Atlassian JIRA (v6.3.4#6332)