Github user darrenfoong commented on the issue:
https://github.com/apache/geode/pull/668
Thanks Jared! Will find time to make the changes and get feedback on the
mailing list.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user darrenfoong commented on the issue:
https://github.com/apache/geode/pull/668
Hi Jared, thank you for your feedback.
The initial use case that I was thinking of involved a user wanting to:
- use Geode with another JDK (which doesn't have the
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/668
Hi Darren,
It looks like `xercesImpl` may need to be declared as a `testCompile`
dependency rather than `testRuntime` in case people are building with a JDK
which does not include
Github user darrenfoong commented on the issue:
https://github.com/apache/geode/pull/668
I've added code to unset/clear the temporarily-set system properties for
testing.
Thank you Kenneth for your feedback.
---
If your project is set up for it, you can reply to this email
Github user darrenfoong commented on the issue:
https://github.com/apache/geode/pull/668
I just realised I'll need to unset the property to prevent any side effects
in the other tests; working on it now.
---
If your project is set up for it, you can reply to this email and have your