[
https://issues.apache.org/jira/browse/HADOOP-15679?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16582097#comment-16582097
]
Steve Loughran commented on HADOOP-15679:
-----------------------------------------
HADOOP-15769 patch 001: timeout for hooks extended and configurable; lots of
tests
* Hook timeout now configurable in core-site through property
"hadoop.service.shutdown.timeout"
* core-default value is 30s, to deal with issue that 10s is too short
* declare interface public/evolving (other services use it too), and mark which
methods are public+stable, which are private test methods
* tests for the shutdown hook manager now actually invokes shutdown invocation
process, verifies ordered and sequential execution of hooks,
including the fact that long sleeping hooks do not hold up successors.
* also test for current registration behaviour of : a second registration is a
no-op, so previous priority and timeout is retained.
I'm happy to declare the default 10s timeout in core-default.xml if people want
that to stay stable; the key thing is that people need to be able to configure
it to larger values
> ShutdownHookManager shutdown time needs to be configurable & extended
> ---------------------------------------------------------------------
>
> Key: HADOOP-15679
> URL: https://issues.apache.org/jira/browse/HADOOP-15679
> Project: Hadoop Common
> Issue Type: Bug
> Components: util
> Affects Versions: 2.8.0, 3.0.0
> Reporter: Steve Loughran
> Assignee: Steve Loughran
> Priority: Major
> Attachments: HADOOP-15679-001.patch
>
>
> HADOOP-12950 added a timeout on shutdowns to avoid problems with hanging
> shutdowns. But the timeout is too short for applications where a large flush
> of data is needed on shutdown.
> A key example of this is Spark apps which save their history to object
> stores, where the file close() call triggers an upload of the final local
> cached block of data (could be 32+MB), and then execute the final mutipart
> commit.
> Proposed
> # make the default sleep time 30s, not 10s
> # make it configurable with a time duration property (with minimum time of
> 1s.?)
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
