albertogpz commented on pull request #697:
URL: https://github.com/apache/geode-native/pull/697#issuecomment-734189634
> > Down to one failed test on a couple of platforms now - so close!
> > ```
> > testXmlCacheCreationWithPools
> > ```
> >
> >
> > Is timing out on Windows and RHEL7.
> > FYI, I'm out this week for the US holiday, so will be of limited, if
any, help. @mmartell & @mreddington can follow up as needed, I'll ping them via
DM.
>
> I managed to fix the test on windows thanks to @gaussianrecurrence . I
hope it is also fixed under RHEL7.
Anyway, I have a question for @mreddington . The test case failed due to the
addition of the following commit:
d2a307d80 cppcache/src/TcrEndpoint.cpp (Matthew Reddington 2020-05-14
17:36:32 -0700 1181) m_cacheImpl->doIfDestroyNotPending([&]()
{
What is the reason for this change? I have not seen it in the development
branch so I wonder if it is really necessary.
----------------------------------------------------------------
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:
[email protected]
