You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2021/03/11 19:10:33 UTC

[GitHub] [geode-native] gaussianrecurrence opened a new pull request #762: GEODE-9026: Fix RegisterKeysTest race condition

gaussianrecurrence opened a new pull request #762:
URL: https://github.com/apache/geode-native/pull/762


    - Sometimes it could happen that in RegisterAllWithConsistencyDisabled
      test the call to afterDestroy happens before the expectance is set.
      THis leads to the test sporiously failing.
    - Expect call has been moved after the listener instantiation to avoid
      this race condition.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [geode-native] pdxcodemonkey merged pull request #762: GEODE-9026: Fix RegisterKeysTest race condition

Posted by GitBox <gi...@apache.org>.
pdxcodemonkey merged pull request #762:
URL: https://github.com/apache/geode-native/pull/762


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [geode-native] rhoughton-pivot commented on pull request #762: GEODE-9026: Fix RegisterKeysTest race condition

Posted by GitBox <gi...@apache.org>.
rhoughton-pivot commented on pull request #762:
URL: https://github.com/apache/geode-native/pull/762#issuecomment-797010227


   The CI change to need Concourse statuses to pass is causing an issue with this PR. We'll work to fix the jobs ASAP so that the PR is mergable.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [geode-native] rhoughton-pivot removed a comment on pull request #762: GEODE-9026: Fix RegisterKeysTest race condition

Posted by GitBox <gi...@apache.org>.
rhoughton-pivot removed a comment on pull request #762:
URL: https://github.com/apache/geode-native/pull/762#issuecomment-797010227


   The CI change to need Concourse statuses to pass is causing an issue with this PR. We'll work to fix the jobs ASAP so that the PR is mergable.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org