Hudson commented on HBASE-20567:

Results for branch master
        [build #334 on 
builds.a.o|https://builds.apache.org/job/HBase%20Nightly/job/master/334/]: (x) 
*{color:red}-1 overall{color}*
details (if available):

(/) {color:green}+1 general checks{color}
-- For more information [see general 

(x) {color:red}-1 jdk8 hadoop2 checks{color}
-- For more information [see jdk8 (hadoop2) 

(x) {color:red}-1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3) 

(/) {color:green}+1 source release artifact{color}
-- See build output for details.

> Pass both old and new descriptors to pre/post hooks of modify operations for 
> table and namespace
> ------------------------------------------------------------------------------------------------
>                 Key: HBASE-20567
>                 URL: https://issues.apache.org/jira/browse/HBASE-20567
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Appy
>            Assignee: Appy
>            Priority: Major
>             Fix For: 3.0.0, 2.1.0, 2.0.1
>         Attachments: HBASE-20567.master.001.patch, 
> HBASE-20567.master.002.patch
> In postModify* hooks like {{postModifyX(..., Descriptor newDesc)}}, there's 
> no way of getting the old descriptor which was there before modification 
> happened.
> Having both old and new descriptors will make the hooks more useful.
> We felt the need when we wanted to audit certain events but there was no way 
> of deducing them by just seeing 'after-state' of modification.
> To keep the method signatures consistent, i have modified both pre and post 
> hooks with new arguments which are well named and commented.

This message was sent by Atlassian JIRA

Reply via email to