[GitHub] [geode-native] pdxcodemonkey commented on pull request #645: GEODE-8469: Enable no-missing-variable-declarations

[GitBox <gi...@apache.org>]

pdxcodemonkey commented on pull request #645:
URL: https://github.com/apache/geode-native/pull/645#issuecomment-697028109


   So the following 3 tests are now consistently failing on Windows:
   ```
   The following tests FAILED:
   	 14 - testRegionAccessThreadSafe (Timeout)
   	 24 - testThinClientCacheableStringArray (Timeout)
   	 84 - testThinClientInterestList2 (Timeout)
   ```
   I did some initial investigation thusly:
   i. I established a baseline on the develop branch for the test testRegionAccessThreadSafe.  On my EC2 instance, it passes in ~75 seconds
   4:00
   ii. I then reverted your changes to just testRegionAccessThreadSafe.cpp , rebuilt and re-ran the test, which timed out
   4:01
   iii.  I reapplied your change to the test, and reverted your changes to just the test framework files /cppcache/integration-test/fw_*.cpp, and re-ran the test, which still timed out
   4:02
   That's about it for low-hanging fruit, I think.  I think we'll have success with a 'bisect' approach, where we revert subsets of the change until it starts passing again


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