[ https://issues.apache.org/jira/browse/HBASE-4271?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13098445#comment-13098445 ]
jirapos...@reviews.apache.org commented on HBASE-4271: ------------------------------------------------------ ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1691/#review1775 ----------------------------------------------------------- Ship it! Looks good Ming! http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/coprocessor/MasterObserver.java <https://reviews.apache.org/r/1691/#comment4052> I think this and the following line should be swapped, but that can be fixed on commit. - Gary On 2011-09-03 01:47:00, Ming Ma wrote: bq. bq. ----------------------------------------------------------- bq. This is an automatically generated e-mail. To reply, visit: bq. https://reviews.apache.org/r/1691/ bq. ----------------------------------------------------------- bq. bq. (Updated 2011-09-03 01:47:00) bq. bq. bq. Review request for hbase, Gary Helmling and Ted Yu. bq. bq. bq. Summary bq. ------- bq. bq. Along with coprocessor work, also fix HBaseAdmin.deleteTable so that it waits until the table is removed from HMaster's HTableDescriptor list. bq. bq. 1. Fix preCreateTable, postCreateTable, preAssign, postAssign, preUnassign APIs. bq. 2. Make coprocessor honor bypass request from preMove, preAddColumn, preModifyColumn, preDeleteColumn. bq. bq. bq. This addresses bug HBASE-4271. bq. https://issues.apache.org/jira/browse/HBASE-4271 bq. bq. bq. Diffs bq. ----- bq. bq. http://svn.apache.org/repos/asf/hbase/trunk/src/test/java/org/apache/hadoop/hbase/coprocessor/TestMasterObserver.java 1164783 bq. http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/MasterCoprocessorHost.java 1164783 bq. http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java 1164783 bq. http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/master/HMaster.java 1164783 bq. http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java 1164783 bq. http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/coprocessor/BaseMasterObserver.java 1164783 bq. http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/coprocessor/MasterObserver.java 1164783 bq. bq. Diff: https://reviews.apache.org/r/1691/diff bq. bq. bq. Testing bq. ------- bq. bq. Unit tests bq. bq. bq. Thanks, bq. bq. Ming bq. bq. > Clean up coprocessor's handlings of table operations > ---------------------------------------------------- > > Key: HBASE-4271 > URL: https://issues.apache.org/jira/browse/HBASE-4271 > Project: HBase > Issue Type: Bug > Reporter: Ming Ma > Assignee: Ming Ma > > Couple fixes we can do w.r.t coprocessor's handlings of table operations. > 1. Honor MasterObserver's requests to bypass default action. > 2. Fix up the function signatures for preCreateTable to use HRegionInfo as > parameter instead. > 3. Invoke postEnableTable, etc. methods after the operations are done. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira