hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Zhijie Shen (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-2629) Make distributed shell use the domain-based timeline ACLs
Date Tue, 07 Oct 2014 23:51:35 GMT

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

Zhijie Shen commented on YARN-2629:
-----------------------------------

bq. 1. I noticed that you've changed a few method signatures, and removed the "throws IOException,
YarnException" clause from the signatures. Meanwhile, in the method I noticed that you're
catching all types of exceptions in one catch statement. I agree that this change saves some
repeated lines of try-catch statements in various call sites, but is it OK catch all exceptions
internally?

Previously, the exception was handled at the caller of publishXXXX(), but I moved into publishXXXX()
to simplify the change, such that there's no need to throw the exception any more. Usually,
it's a good behavior to handle each potential exception explicitly. Here since I intentionally
don't want to do such fine-grained handling, I choose to log whatever the exception is.

bq. 2. Maybe we want to add one more test with no arguments on domain/view_acl/modify_acl?
Timeline domain is a newly added feature, and maybe we'd like to make sure it would not break
any existing distributed shell usages (esp. in some scripts)?

Added one

bq. This variable name looks unclear. Maybe we want to rename it so that people can easily
connect it with timeline domains?

It's explained in the usage of the option and have be commented:
{code}
  // Flag to indicate whether to create the domain of the given ID
  private boolean toCreate = false;
{code}

but let me elaborate this var in code.

> Make distributed shell use the domain-based timeline ACLs
> ---------------------------------------------------------
>
>                 Key: YARN-2629
>                 URL: https://issues.apache.org/jira/browse/YARN-2629
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: timelineserver
>            Reporter: Zhijie Shen
>            Assignee: Zhijie Shen
>         Attachments: YARN-2629.1.patch, YARN-2629.2.patch, YARN-2629.3.patch
>
>
> For demonstration the usage of this feature (YARN-2102), it's good to make the distributed
shell create the domain, and post its timeline entities into this private space.



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

Mime
View raw message