hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Varun Saxena (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-4224) Support fetching entities by UID and change the REST interface to conform to current REST APIs' in YARN
Date Tue, 26 Jan 2016 16:39:39 GMT

    [ https://issues.apache.org/jira/browse/YARN-4224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15117526#comment-15117526
] 

Varun Saxena commented on YARN-4224:
------------------------------------

Thanks [~sjlee0] for the review.

Regarding the comments,

bq. Comments regarding using primitive long vs Long.
I have used Long for a reason here. I plan to use the class TimelineReaderContext while fixing
YARN-4446(which is regarding refactoring code to reduce number of params in reader API). In
reader API, flow run id being null indicates that it has not come from the client. Probably
we can use a sentinel value like -1 and use primitive long as well(assuming run id wont be
negative most probably) but current reader code assumes null indicating flow run has not been
supplied by client. 
Thoughts ?

bq. Comments regarding class and method visibility.
Agree mostly. But shouldn't we make TimelineReaderUtils public(after moving web services related
methods as per Li's comments to a new class). Cant say where but split and joinAndEscapeStrings
methods might be useful elsewhere in future. Look somewhat generic. Thoughts ?

bq. redundant checks in equals.
Agree. Will fix it.

bq. We shouldn't use Throwable.printStackTrace() which goes to standard err console
Left it by mistake. Was using it to debug some unit test case failure. Will fix it.

> Support fetching entities by UID and change the REST interface to conform to current
REST APIs' in YARN
> -------------------------------------------------------------------------------------------------------
>
>                 Key: YARN-4224
>                 URL: https://issues.apache.org/jira/browse/YARN-4224
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: timelineserver
>    Affects Versions: YARN-2928
>            Reporter: Varun Saxena
>            Assignee: Varun Saxena
>              Labels: yarn-2928-1st-milestone
>         Attachments: YARN-4224-YARN-2928.01.patch, YARN-4224-YARN-2928.05.patch, YARN-4224-feature-YARN-2928.04.patch,
YARN-4224-feature-YARN-2928.wip.02.patch, YARN-4224-feature-YARN-2928.wip.03.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message