jdeppe-pivotal opened a new pull request #7192:
URL: https://github.com/apache/geode/pull/7192


   - This fix also addresses GEODE-9428 where CLUSTERDOWN is occasionally
     reported.
   - The NativeRedisClusterTestRule now checks each node to make sure they
     are all reporting the correct node count instead of just one. Nodes
     are not always all ready at once.
   - When processing CLUSTER INFO sometimes a 'master' reports with no
     slots which indicates it is transitioning or will become a replica.
     Originally we would mark such a node as a replica even before it had
     fully transitioned. However, this would cause problems since it then
     resulted in a too early report that the cluster was ready.
   
   <!-- Thank you for submitting a contribution to Apache Geode. -->
   
   <!-- In order to streamline the review of the contribution we ask you
   to ensure the following steps have been taken: 
   -->
   
   ### For all changes:
   - [ ] Is there a JIRA ticket associated with this PR? Is it referenced in 
the commit message?
   
   - [ ] Has your PR been rebased against the latest commit within the target 
branch (typically `develop`)?
   
   - [ ] Is your initial contribution a single, squashed commit?
   
   - [ ] Does `gradlew build` run cleanly?
   
   - [ ] Have you written or updated unit tests to verify your changes?
   
   - [ ] If adding new dependencies to the code, are these dependencies 
licensed in a way that is compatible for inclusion under [ASF 
2.0](http://www.apache.org/legal/resolved.html#category-a)?
   
   <!-- Note:
   Please ensure that once the PR is submitted, check Concourse for build 
issues and
   submit an update to your PR as soon as possible. If you need help, please 
send an
   email to [email protected].
   -->
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to