You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by "Jason Huynh (JIRA)" <ji...@apache.org> on 2017/05/06 00:04:04 UTC

[jira] [Created] (GEODE-2890) Incorrect debug log location in AbstractGatewaySenderEventProcessor.processQueue()

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)