hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "stack (JIRA)" <j...@apache.org>
Subject [jira] Updated: (HBASE-2669) HCM.shutdownHook causes data loss with hbase.client.write.buffer != 0
Date Sat, 05 Jun 2010 20:33:53 GMT

     [ https://issues.apache.org/jira/browse/HBASE-2669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

stack updated HBASE-2669:
-------------------------

    Fix Version/s: 0.21.0
                       (was: 0.20.5)

Moving out of 0.20.5.  There is a workaround for now.  See suppressHdfsShutdownHook in HRegionServer
for how to remove the installed HCM shutdown hook and install your own instead... running
the HCM one after yours.   Its ugly but it works.  While the implication is dataloss if you
SIGTERM the hosting application, this is sort of a new request/feature, or putting it another
way, its a problem we've had for a long time, not particular to 0.20.5.  I don't thnk it a
blocker.  Sink the coming RC if you think otherwise Benoit by voting against it but for now,
I'd like to show there is movement around 0.20.5 release by posting a new RC. 

> HCM.shutdownHook causes data loss with hbase.client.write.buffer != 0
> ---------------------------------------------------------------------
>
>                 Key: HBASE-2669
>                 URL: https://issues.apache.org/jira/browse/HBASE-2669
>             Project: HBase
>          Issue Type: Bug
>          Components: client
>            Reporter: Benoit Sigoure
>            Assignee: Benoit Sigoure
>            Priority: Critical
>             Fix For: 0.21.0
>
>
> In my application I set {{hbase.client.write.buffer}} to a reasonably small value (roughly
64 edits) in order to try to batch a few {{Put}} together before talking to HBase.  When my
application does a graceful shutdown, I call {{HTable#flushCommits}} in order to flush any
pending change to HBase.  I want to do the same thing when I get a {{SIGTERM}} by using {{Runtime#addShutdownHook}}
but this is impossible since {{HConnectionManager}} already registers a shutdown hook that
invokes {{HConnectionManager#deleteAllConnections}}.  This static method closes all the connections
to HBase and then all connections to ZooKeeper.  Because all shutdown hooks run in parallel,
my hook will attempt to flush edits while connections are getting closed.
> There is no way to guarantee the order in which the hooks will execute, so I propose
that we remove the hook in the HCM altogether and provide some user-visible API they call
in their own hook after they're done flushing their stuff, if they really want to do a graceful
shutdown.  I expect that a lot of users won't use a hook though, otherwise this issue would
have cropped up already.  For those users, connections won't get "gracefully" terminated,
but I don't think that would be a problem since the underlying TCP socket will get closed
by the OS anyway, so things like ZooKeeper and such should realize that the connection has
been terminated and assume the client is gone, and do the necessary clean-up on their side.
> An alternate fix would be to leave the hook in place by default but keep a reference
to it and add a user-visible API to be able to un-register the hook.  I find this ugly.
> Thoughts?

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message