Jason Huynh created GEODE-2890:
----------------------------------

             Summary: Incorrect debug log location in 
AbstractGatewaySenderEventProcessor.processQueue() 
                 Key: GEODE-2890
                 URL: https://issues.apache.org/jira/browse/GEODE-2890
             Project: Geode
          Issue Type: Bug
          Components: wan
            Reporter: Jason Huynh


The following code snippet in processQueue() for AEQ's appears to be outside of 
an if condition where we check to see if the node is primary still or not.  
This line prints for every event and I believe it should be inside the previous 
if condition.     

{noformat}
if (qpr != null) {
   BucketRegion bucket = qpr.getDataStore().getLocalBucketById(bucketId);
   if (bucket == null || !bucket.getBucketAdvisor().isPrimary()) {
     event.setPossibleDuplicate(true);
    //I think the debug log should be placed here?
   }
}
if (isDebugEnabled) {
  logger.debug( "Bucket id: {} is no longer primary on this node. The event {} 
will be dispatched from this node with possibleDuplicate set to true.", 
bucketId, event);
}
{noformat}





--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to