[GitHub] geode issue #474: GEODE-2788: Add official Socket timeout parameter when con...

2017-06-07 Thread kohlmu-pivotal
Github user kohlmu-pivotal commented on the issue: https://github.com/apache/geode/pull/474 @masaki-yamakawa I'm running precheckin. If it passes I'll merge this PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] geode issue #474: GEODE-2788: Add official Socket timeout parameter when con...

2017-06-06 Thread masaki-yamakawa
Github user masaki-yamakawa commented on the issue: https://github.com/apache/geode/pull/474 @galen-pivotal Thank you very much. Because there was a conflicting file with the change so far, I merged again, then pushed. Travis-CI worked normally. --- If your project is set up for

[GitHub] geode issue #474: GEODE-2788: Add official Socket timeout parameter when con...

2017-06-05 Thread galen-pivotal
Github user galen-pivotal commented on the issue: https://github.com/apache/geode/pull/474 @masaki-yamakawa [Stack overflow](https://stackoverflow.com/questions/17606874/trigger-a-travis-ci-rebuild-without-pushing-a-commit) suggests that you can trigger a build by closing and

[GitHub] geode issue #474: GEODE-2788: Add official Socket timeout parameter when con...

2017-05-26 Thread masaki-yamakawa
Github user masaki-yamakawa commented on the issue: https://github.com/apache/geode/pull/474 @galen-pivotal Thank you very much. I confirmed the message sent to the dev mailing list. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] geode issue #474: GEODE-2788: Add official Socket timeout parameter when con...

2017-05-26 Thread galen-pivotal
Github user galen-pivotal commented on the issue: https://github.com/apache/geode/pull/474 @masaki-yamakawa Looks like Travis doesn't think the build has started. I've sent a message to the developer list; someone there should have an idea what the issue is. --- If your project is

[GitHub] geode issue #474: GEODE-2788: Add official Socket timeout parameter when con...

2017-05-26 Thread masaki-yamakawa
Github user masaki-yamakawa commented on the issue: https://github.com/apache/geode/pull/474 I am sorry, once I rebase to confirm the build, travis-ci hangs for some reason. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] geode issue #474: GEODE-2788: Add official Socket timeout parameter when con...

2017-05-21 Thread masaki-yamakawa
Github user masaki-yamakawa commented on the issue: https://github.com/apache/geode/pull/474 I fixed LocatorLauncherTest as follows and tested that BUILD succeeded. ```java @BeforeClass public static void caseSetUp() { InternalDistributedSystem ds =

[GitHub] geode issue #474: GEODE-2788: Add official Socket timeout parameter when con...

2017-05-21 Thread masaki-yamakawa
Github user masaki-yamakawa commented on the issue: https://github.com/apache/geode/pull/474 @bschuchardt Thank you for pointing it out. I resolved the conflicts. Travis CI's BUILD failed, but I think it is the effect of GEODE-2959 change. --- If your project is set up for

[GitHub] geode issue #474: GEODE-2788: Add official Socket timeout parameter when con...

2017-05-18 Thread bschuchardt
Github user bschuchardt commented on the issue: https://github.com/apache/geode/pull/474 Masaki, it looks like there are some conflicts that need to be resolved --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] geode issue #474: GEODE-2788: Add official Socket timeout parameter when con...

2017-05-18 Thread masaki-yamakawa
Github user masaki-yamakawa commented on the issue: https://github.com/apache/geode/pull/474 I fixed the test that was failing with precheckin. As for the modification of XSD, I think that there is no conclusion yet, but at the present time, I am dealing with assumption to change

[GitHub] geode issue #474: GEODE-2788: Add official Socket timeout parameter when con...

2017-05-10 Thread bschuchardt
Github user bschuchardt commented on the issue: https://github.com/apache/geode/pull/474 Masaki, this change set is breaking unit tests because the new XSD attribute is being put into XML files generated for older versions. CacheXml80DUnitTest, for instance, is breaking. l80Please

[GitHub] geode issue #474: GEODE-2788: Add official Socket timeout parameter when con...

2017-05-10 Thread bschuchardt
Github user bschuchardt commented on the issue: https://github.com/apache/geode/pull/474 I am running precheckin on this PR and will merge it to develop afterward. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your