[
https://issues.apache.org/jira/browse/HBASE-1730?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13091895#comment-13091895
]
jirapos...@reviews.apache.org commented on HBASE-1730:
------------------------------------------------------
bq. On 2011-08-18 23:15:10, Michael Stack wrote:
bq. > src/main/java/org/apache/hadoop/hbase/ipc/HMasterInterface.java, line 80
bq. > <https://reviews.apache.org/r/1479/diff/1/?file=32327#file32327line80>
bq. >
bq. > Javadoc does not agree w/ what is being actually returned.
fixed this.
bq. On 2011-08-18 23:15:10, Michael Stack wrote:
bq. > src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java,
line 1498
bq. > <https://reviews.apache.org/r/1479/diff/1/?file=32328#file32328line1498>
bq. >
bq. > Why are we waiting between unassigns? To lessen load on cluster?
yes.
bq. On 2011-08-18 23:15:10, Michael Stack wrote:
bq. > src/main/java/org/apache/hadoop/hbase/master/HMaster.java, line 1067
bq. > <https://reviews.apache.org/r/1479/diff/1/?file=32330#file32330line1067>
bq. >
bq. > Is there anything in here to stop someone disabling a table or
preventing the load balancer moving regions while this is all going on?
the problem with this is, if you run more than one alter concurrently, we do
not have a way to restore the state of the loadbalancer.
bq. On 2011-08-18 23:15:10, Michael Stack wrote:
bq. >
src/main/java/org/apache/hadoop/hbase/master/handler/ClosedRegionHandler.java,
line 101
bq. > <https://reviews.apache.org/r/1479/diff/1/?file=32332#file32332line101>
bq. >
bq. > This looks like you are fixing a bug we have in hbase?
No. this is required for the bulk reopen operation and impacts only reopen
datastructures.
- Nileema
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1479/#review1535
-----------------------------------------------------------
On 2011-08-26 17:25:21, Nileema Shingte wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/1479/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2011-08-26 17:25:21)
bq.
bq.
bq. Review request for hbase, Dhruba Borthakur, Ted Yu, Michael Stack, and
Jonathan Gray.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. When the master receives an alter table call (addColumn, modifyColumn,
deleteColumn, modifyTable), it updates the .tableinfo and then closes all the
regions of that table. The patch includes:
bq.
bq. 1. Changes to reopen the regions when any of the above operations are
performed.
bq. 2. Best effort is made to preserve the locality of regions by assigning it
a region plan before closing it.
bq. 3. Throttling logic that ensures that only a configurable number of
regions are closed per region server at a time.
bq. 4. alter command in the hbase shell will block until all the regions are
updated, providing a status "x/y regions updated" every second.
bq. 5. alter_async command that works exactly like alter, except that it does
not block for completion or provide the status.
bq. 6. alter_status <table_name> which is a sync call and blocks to provide
the "x/y regions updated" status per second until all regions are updated.
bq. 7. modification in the unit test for enabling alter without disabling the
table.
bq.
bq.
bq. This addresses bug HBASE-1730.
bq. https://issues.apache.org/jira/browse/HBASE-1730
bq.
bq.
bq. Diffs
bq. -----
bq.
bq. src/test/java/org/apache/hadoop/hbase/client/TestAdmin.java 5965919
bq. src/main/ruby/shell.rb 1ec330f
bq. src/main/ruby/shell/commands/alter.rb 1dd43ad
bq. src/main/ruby/shell/commands/alter_async.rb PRE-CREATION
bq. src/main/ruby/shell/commands/alter_status.rb PRE-CREATION
bq.
src/main/java/org/apache/hadoop/hbase/master/handler/TableEventHandler.java
09891aa
bq. src/main/ruby/hbase/admin.rb 4460d6e
bq. src/main/java/org/apache/hadoop/hbase/master/HMaster.java 9784195
bq.
src/main/java/org/apache/hadoop/hbase/master/handler/ClosedRegionHandler.java
ae43837
bq.
src/main/java/org/apache/hadoop/hbase/master/handler/TableAddFamilyHandler.java
6332953
bq. src/main/java/org/apache/hadoop/hbase/master/BulkReOpen.java
PRE-CREATION
bq. src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java b5d65ce
bq. src/main/java/org/apache/hadoop/hbase/executor/EventHandler.java 0c9ecce
bq. src/main/java/org/apache/hadoop/hbase/ipc/HMasterInterface.java c0aa024
bq. src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
49d1e7c
bq.
bq. Diff: https://reviews.apache.org/r/1479/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq. I am putting this up for initial review. I have tested the functionality
in a pseudo distributed mode.
bq. Need to run unit tests.
bq.
bq.
bq. Thanks,
bq.
bq. Nileema
bq.
bq.
> Near-instantaneous online schema and table state updates
> --------------------------------------------------------
>
> Key: HBASE-1730
> URL: https://issues.apache.org/jira/browse/HBASE-1730
> Project: HBase
> Issue Type: Improvement
> Reporter: Andrew Purtell
> Assignee: stack
> Priority: Critical
> Fix For: 0.92.0
>
> Attachments: 1730-v2.patch, 1730-v3.patch, 1730.patch,
> HBASE-1730.patch
>
>
> We should not need to take a table offline to update HCD or HTD.
> One option for that is putting HTDs and HCDs up into ZK, with mirror on disk
> catalog tables to be used only for cold init scenarios, as discussed on IRC.
> In this scheme, regionservers hosting regions of a table would watch
> permanent nodes in ZK associated with that table for schema updates and take
> appropriate actions out of the watcher. In effect, schema updates become
> another item in the ToDo list.
> {{/hbase/tables/<table-name>/schema}}
> Must be associated with a write locking scheme also handled with ZK
> primitives to avoid situations where one concurrent update clobbers another.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
