[GitHub] geode issue #647: GEODE-3271: Refactor WanCommands

2017-07-21 Thread YehEmily
Github user YehEmily commented on the issue: https://github.com/apache/geode/pull/647 @PurelyApplied Updated! Thanks for the feedback! --- 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 project does not have this fea

[GitHub] geode issue #647: GEODE-3271: Refactor WanCommands

2017-07-21 Thread PurelyApplied
Github user PurelyApplied commented on the issue: https://github.com/apache/geode/pull/647 +1 as it stands. Unimportant nitpicks, rambling observations, and "it could be even better if...": * `punePort` appears in these tests a lot. I have no idea what `pune` is sup