[
https://issues.apache.org/jira/browse/HBASE-3276?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12936119#action_12936119
]
HBase Review Board commented on HBASE-3276:
-------------------------------------------
Message from: "Ryan Rawson" <[email protected]>
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://review.cloudera.org/r/1252/#review1993
-----------------------------------------------------------
trunk/src/main/java/org/apache/hadoop/hbase/KeyValue.java
<http://review.cloudera.org/r/1252/#comment6297>
what are all the consequences for not sorting by type when using
KVComparator? Does this mean we might create HFiles that not sorted properly,
because the HFile comparator uses the KeyComparator directly with ignoreType =
false.
While in memstore we can rely on memstoreTS to roughly order by insertion
time, and the Put/Delete should probably work in that situation, you are
talking about modifiying a pretty core and important concept in how we sort
things.
There are other ways to reconcile bugs like this, one of them is to extend
the memstoreTS concept into the HFile and use that to reconcile during reads.
There is another JIRA where I proposed this.
If we are talking about 0.92 and beyond I'd prefer building a solid base
rather than dangerous hacks like this. Our unit tests are not extremely
extensive, so while they might pass, that doesnt guarantee lack of bad
behaviour later on.
- Ryan
> delete followed by a put with the same timestamp
> ------------------------------------------------
>
> Key: HBASE-3276
> URL: https://issues.apache.org/jira/browse/HBASE-3276
> Project: HBase
> Issue Type: Bug
> Reporter: Kannan Muthukkaruppan
> Assignee: Kannan Muthukkaruppan
>
> [Note: This issue is relevant only for cases that don't use the default
> "time" based versions, but provide/manage versions explicitly.]
> The fix for HBASE-1485 ensures that if there are multiple puts with the same
> timestamp the later one wins.
> However, if there is a delete for a specific timestamp, then the later put
> doesn't win.
> Say for example the following is the sequence of operations:
> put row/col/v1 - value1
> deleteColumn row/col/v1
> put row/col/v1 - value2
> Without the deleteColumn(), HBASE-1485 ensures that "value2" is the winner.
> However, with the deleteColumn() thrown into the mix, the delete wins, and
> one cannot insert a new value at that version. [The only, unsatisfactory,
> workaround at this point seems to be trigger a major compaction. The major
> compact would clear the delete marker, and allow new cells to be created with
> that version again.]
> ---
> Seems like it might not be too complicated to extend the fix for HBASE-1485
> to also respect ordering between delete/put operations. I'll look into this
> further.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
