I filed a separate JIRA for this as there is no geode.jar and 
KillJavaProcesses() will not work correctly.

> On 8 Feb, 2017, at 14:47, dgkimura <g...@git.apache.org> wrote:
> 
> Github user dgkimura commented on a diff in the pull request:
> 
>    https://github.com/apache/geode/pull/391#discussion_r100191055
> 
>    --- Diff: src/clicache/integration-test/CacheHelperN.cs ---
>    @@ -2389,7 +2389,7 @@ public static void KillJavaProcesses()
>               string commandline = item["CommandLine"].ToString();
> 
>               Util.Log("processId:{0} name:{1}", item["ProcessId"], 
> item["Name"]);
>    -          if (commandline.Contains("geode.jar"))
>    +          if (commandline.Contains("gemfire.jar"))
>    --- End diff --
> 
>    I'm curious why this one you're switching geode with gemfire.  What makes 
> this one different?
> 
> 
> ---
> 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 feature
> enabled and wishes so, or if the feature is enabled but not working, please
> contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
> with INFRA.
> ---

Reply via email to