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

Hudson commented on HBASE-10069:
--------------------------------

FAILURE: Integrated in HBase-0.98 #8 (See 
[https://builds.apache.org/job/HBase-0.98/8/])
HBASE-10069 Potential duplicate calls to log#appendNoSync() in 
HRegion#doMiniBatchMutation() (sershe: rev 1550237)
* 
/hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegion.java


> Potential duplicate calls to log#appendNoSync() in 
> HRegion#doMiniBatchMutation()
> --------------------------------------------------------------------------------
>
>                 Key: HBASE-10069
>                 URL: https://issues.apache.org/jira/browse/HBASE-10069
>             Project: HBase
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee: Sergey Shelukhin
>            Priority: Minor
>             Fix For: 0.98.0, 0.99.0
>
>         Attachments: HBASE-10069.patch
>
>
> In HRegion#doMiniBatchMutation():
> {code}
>         if (nonceGroup != currentNonceGroup || nonce != currentNonce) {
>           if (walEdit.size() > 0) {
>             assert isInReplay;
>             txid = this.log.appendNoSync(this.getRegionInfo(), 
> htableDescriptor.getTableName(),
>                   walEdit, m.getClusterIds(), now, htableDescriptor, 
> this.sequenceId, true,
>                   currentNonceGroup, currentNonce);
>             hasWalAppends = true;
>           }
>           currentNonceGroup = nonceGroup;
>           currentNonce = nonce;
>         }
>         // Add WAL edits by CP
>         WALEdit fromCP = batchOp.walEditsFromCoprocessors[i];
>         if (fromCP != null) {
>           for (KeyValue kv : fromCP.getKeyValues()) {
>             walEdit.add(kv);
>           }
>         }
> ...
>       Mutation mutation = batchOp.getMutation(firstIndex);
>       if (walEdit.size() > 0) {
>         txid = this.log.appendNoSync(this.getRegionInfo(), 
> this.htableDescriptor.getTableName(),
>               walEdit, mutation.getClusterIds(), now, this.htableDescriptor, 
> this.sequenceId,
>               true, currentNonceGroup, currentNonce);
>         hasWalAppends = true;
>       }
> {code}
> If fromCP is null, there may not be new edits added to walEdit.
> But log#appendNoSync() would be called one more time at line 2368.



--
This message was sent by Atlassian JIRA
(v6.1.4#6159)

Reply via email to