ramkrishna.s.vasudevan created HBASE-15259:
----------------------------------------------

             Summary: WALEdits under replay will also be replicated
                 Key: HBASE-15259
                 URL: https://issues.apache.org/jira/browse/HBASE-15259
             Project: HBase
          Issue Type: Bug
            Reporter: ramkrishna.s.vasudevan
            Priority: Minor


I need to verify this but seeing the code
{code}
try {
        // We are about to append this edit; update the region-scoped sequence 
number.  Do it
        // here inside this single appending/writing thread.  Events are 
ordered on the ringbuffer
        // so region sequenceids will also be in order.
        regionSequenceId = entry.stampRegionSequenceId();
        // Edits are empty, there is nothing to append.  Maybe empty when we 
are looking for a
        // region sequence id only, a region edit/sequence id that is not 
associated with an actual
        // edit. It has to go through all the rigmarole to be sure we have the 
right ordering.
        if (entry.getEdit().isEmpty()) {
          return;
        }

        // Coprocessor hook.
        if (!coprocessorHost.preWALWrite(entry.getHRegionInfo(), entry.getKey(),
            entry.getEdit())) {
          if (entry.getEdit().isReplay()) {
            // Set replication scope null so that this won't be replicated
            entry.getKey().setScopes(null);
          }
        }
        if (!listeners.isEmpty()) {
          for (WALActionsListener i: listeners) {
            // TODO: Why does listener take a table description and CPs take a 
regioninfo?  Fix.
            i.visitLogEntryBeforeWrite(entry.getHTableDescriptor(), 
entry.getKey(),
              entry.getEdit());
          }
        }
{code}
When a WALEdit is in replay we set the Logkey to null. But in the 
visitLogEntryBeforeWrite() we again set the LogKey based on the replication 
scope associated with the cells. So the previous step of setting null does not 
work here?



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to