[ https://issues.apache.org/jira/browse/HBASE-20567?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16477602#comment-16477602 ]
Mike Drob commented on HBASE-20567: ----------------------------------- +1 on v2, can fix checkstyle indentation on commit. (lmk if you want me to take care of it, [~appy]) > 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 > 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 (v7.6.3#76005)