[ 
https://issues.apache.org/jira/browse/OMID-84?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16354342#comment-16354342
 ] 

ASF GitHub Bot commented on OMID-84:
------------------------------------

Github user ebortnikov commented on a diff in the pull request:

    https://github.com/apache/incubator-omid/pull/19#discussion_r166405079
  
    --- Diff: 
hbase-client/src/main/java/org/apache/omid/transaction/HBaseTransaction.java ---
    @@ -31,25 +31,31 @@
     public class HBaseTransaction extends AbstractTransaction<HBaseCellId> {
         private static final Logger LOG = 
LoggerFactory.getLogger(HBaseTransaction.class);
     
    -    public HBaseTransaction(long transactionId, long epoch, 
Set<HBaseCellId> writeSet, AbstractTransactionManager tm) {
    -        super(transactionId, epoch, writeSet, tm);
    +    public HBaseTransaction(long transactionId, long epoch, 
Set<HBaseCellId> writeSet, Set<HBaseCellId> conflictFreeWriteSet, 
AbstractTransactionManager tm) {
    +        super(transactionId, epoch, writeSet, conflictFreeWriteSet, tm);
         }
     
    -    public HBaseTransaction(long transactionId, long readTimestamp, 
VisibilityLevel visibilityLevel, long epoch, Set<HBaseCellId> writeSet, 
AbstractTransactionManager tm) {
    -        super(transactionId, readTimestamp, visibilityLevel, epoch, 
writeSet, tm);
    +    public HBaseTransaction(long transactionId, long readTimestamp, 
VisibilityLevel visibilityLevel, long epoch, Set<HBaseCellId> writeSet, 
Set<HBaseCellId> conflictFreeWriteSet, AbstractTransactionManager tm) {
    +        super(transactionId, readTimestamp, visibilityLevel, epoch, 
writeSet, conflictFreeWriteSet, tm);
         }
     
    +    private void cleanCell(HBaseCellId cell) {
    --- End diff --
    
    cleanCell --> deleteCell? 


> Conflict free writes
> --------------------
>
>                 Key: OMID-84
>                 URL: https://issues.apache.org/jira/browse/OMID-84
>             Project: Apache Omid
>          Issue Type: Sub-task
>            Reporter: Ohad Shacham
>            Assignee: Ohad Shacham
>            Priority: Major
>
> Today, all the writes done by a transaction are taking part in conflict 
> analysis. The purpose of this feature is to let the user decide for each 
> write, whether it should take part in the conflict analysis. 
> The motivation infers from Apache Phoenix that utilizes this feature when 
> writing to the secondary index and also when writing to the data table for 
> immutable tables (each key is added once and is not modified).



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to