Skip to content

Conversation

@nabarunnag
Copy link
Contributor

GEODE-1032 : Placed additional wait time to check for empty queue, refactored WANTestBase.java to remove unused functions, replaced wait criterions with awaitility.

  • Added additional wait time in awaitility to check for empty queue [WANTestBase.java]
  • Replaced wait criterion with awaitility [WANTestBase.java]
  • Removed used parameters from function signatures [WANTestBase.java]
  • Added listener to put the thread to sleep on create event so that the transmission is slowed and sender queue is not empty when the cache is closed.[testReplicatedSerialPropagationWithRemoteReceiverRestartedOnOtherNode]
  • Used the new function signature for validateRegionSize which passes time to wait as a parameter for test cases which wait for 240 seconds.

…factored WANTestBase.java to remove unused functions, replaced wait criterions with awaitility.

* Added additional wait time in awaitility to check for empty queue [WANTestBase.java]
* Replaced wait criterion  with awaitility [WANTestBase.java]
* Removed used parameters from function signatures [WANTestBase.java]
* Added listener to put the thread to sleep on create event so that the transmission is slowed and sender queue is not empty when the cache is closed.[testReplicatedSerialPropagationWithRemoteReceiverRestartedOnOtherNode]
* Used the new function signature for validateRegionSize which passes time to wait as a parameter for test cases which wait for 240 seconds.
@asfgit asfgit closed this in 84a4b47 Apr 18, 2016
bschuchardt pushed a commit to bschuchardt/geode that referenced this pull request Jul 10, 2020
The benchmarks we not logging any of the geode logs, they reported "ERROR
StatusLogger No Log4j 2 configuration file found."

I believe this was broken when we switched to having geode-log4j as a separate
module. Adding that module in as a dependency, which reenables standard geode
logging.
robbadler pushed a commit to smgoller/geode that referenced this pull request Jul 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant