[
https://issues.apache.org/jira/browse/HBASE-2399?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13084493#comment-13084493
]
[email protected] commented on HBASE-2399:
------------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1484/#review1437
-----------------------------------------------------------
Ship it!
http://svn.apache.org/repos/asf/hbase/trunk/src/test/java/org/apache/hadoop/hbase/client/TestAdmin.java
<https://reviews.apache.org/r/1484/#comment3314>
Remove whitespace.
http://svn.apache.org/repos/asf/hbase/trunk/src/test/java/org/apache/hadoop/hbase/client/TestAdmin.java
<https://reviews.apache.org/r/1484/#comment3316>
This was used for debugging, right ?
We may not need it.
- Ted
On 2011-08-12 22:58:55, Ming Ma wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/1484/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2011-08-12 22:58:55)
bq.
bq.
bq. Review request for hbase.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. 1. Add tests for forcesplit multi-column-family scenarios.
bq. 2. Modify HRegion so that it picks splitpoint based on largest store,
instead of the first splittable store. It applies to both forcesplit and
automatic split.
bq.
bq.
bq. This addresses bug hbase-2399.
bq. https://issues.apache.org/jira/browse/hbase-2399
bq.
bq.
bq. Diffs
bq. -----
bq.
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java
1157283
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/test/java/org/apache/hadoop/hbase/HBaseTestingUtility.java
1157283
bq.
http://svn.apache.org/repos/asf/hbase/trunk/src/test/java/org/apache/hadoop/hbase/client/TestAdmin.java
1157283
bq.
bq. Diff: https://reviews.apache.org/r/1484/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq.
bq. Thanks,
bq.
bq. Ming
bq.
bq.
> Forced splits only act on the first family in a table
> -----------------------------------------------------
>
> Key: HBASE-2399
> URL: https://issues.apache.org/jira/browse/HBASE-2399
> Project: HBase
> Issue Type: Bug
> Components: regionserver
> Affects Versions: 0.20.3
> Reporter: Jonathan Gray
> Assignee: Jonathan Gray
> Priority: Critical
> Labels: moved_from_0_20_5
> Fix For: 0.92.0
>
> Attachments: HBASE-2399-test-v1.patch
>
>
> While working on a patch for HBASE-2375, I came across a few bugs in the
> existing code related to splits.
> If a user triggers a manual split, it flips a forceSplit boolean to true and
> then triggers a compaction (this is very similar to my current implementation
> for HBASE-2375). However, the forceSplit boolean is flipped back to false at
> the beginning of Store.compact(). So the force split only acts on the first
> family in the table. If that Store is not splittable for some reason (it is
> empty or has only one row), then the entire region will not be split,
> regardless of what is in other families.
> Even if there is data in the first family, the midKey is determined based
> solely on that family. If it has two rows and the next family has 1M rows,
> we pick the split key based on the two rows.
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
