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-6342) Issues in async API of TimelineClient
Date Wed, 15 Mar 2017 11:05:41 GMT

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

Varun Saxena commented on YARN-6342:
------------------------------------

bq. In stop: it calls shutdownNow which doens't wait for pending tasks, should it use shutdown
instead ?
The executor we use in TimelineV2ClientImpl is a single thread executor with only one task
i.e. the runnable created by createRunnable(). So there wont be any pending tasks. And inside
createRunnable()'s run method, in finally block, we try to dispatch as many events as we can
in 2 seconds.

bq. If any exception happens when publish one entity (publishWithoutBlockingOnQueue), the
thread exists
This makes sense. I think we should catch the exception so that thread does not exit.

> Issues in async API of TimelineClient
> -------------------------------------
>
>                 Key: YARN-6342
>                 URL: https://issues.apache.org/jira/browse/YARN-6342
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Jian He
>
> Found these with [~rohithsharma] while browsing the code
> - In stop: it calls shutdownNow which doens't wait for pending tasks, should it use shutdown
instead ?
> {code}
>     public void stop() {
>       LOG.info("Stopping TimelineClient.");
>       executor.shutdownNow();
>       try {
>         executor.awaitTermination(DRAIN_TIME_PERIOD, TimeUnit.MILLISECONDS);
>       } catch (InterruptedException e) {
> {code}
> - In TimelineClientImpl#createRunnable:
> If any exception happens when publish one entity (publishWithoutBlockingOnQueue), the
thread exists. I think it should try best effort to continue publishing the timeline entities,
one failure should not cause all followup entities not published.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

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


Mime
View raw message