Josh Elser created HBASE-19953:
----------------------------------

             Summary: Avoid calling post* hook when procedure fails
                 Key: HBASE-19953
                 URL: https://issues.apache.org/jira/browse/HBASE-19953
             Project: HBase
          Issue Type: Bug
          Components: master, proc-v2
            Reporter: Ramesh Mani
            Assignee: Josh Elser
             Fix For: 2.0.0-beta-2


Ramesh pointed out a case where I think we're mishandling some post\* 
MasterObserver hooks. Specifically, I'm looking at the deleteNamespace.

We synchronously execute the DeleteNamespace procedure. When the user provides 
a namespace that isn't empty, the procedure does a rollback (which is just a 
no-op), but this doesn't propagate an exception up to the 
NonceProcedureRunnable in {{HMaster#deleteNamespace}}. It took Ramesh pointing 
it out a bit better to me that the code executes a bit differently than we 
actually expect.

I think we need to double-check our post hooks and make sure we aren't invoking 
them when the procedure actually failed. cc/ [~Apache9], [~stack].



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to