[ 
https://issues.apache.org/jira/browse/GEODE-3513?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16140925#comment-16140925
 ] 

ASF GitHub Bot commented on GEODE-3513:
---------------------------------------

Github user upthewaterspout commented on a diff in the pull request:

    https://github.com/apache/geode/pull/740#discussion_r135151782
  
    --- Diff: 
geode-assembly/src/test/java/org/apache/geode/session/tests/Jetty9CachingClientServerTest.java
 ---
    @@ -0,0 +1,83 @@
    +/*
    + * Licensed to the Apache Software Foundation (ASF) under one or more 
contributor license
    + * agreements. See the NOTICE file distributed with this work for 
additional information regarding
    + * copyright ownership. The ASF licenses this file to You under the Apache 
License, Version 2.0 (the
    + * "License"); you may not use this file except in compliance with the 
License. You may obtain a
    + * copy of the License at
    + *
    + * http://www.apache.org/licenses/LICENSE-2.0
    + *
    + * Unless required by applicable law or agreed to in writing, software 
distributed under the License
    + * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF 
ANY KIND, either express
    + * or implied. See the License for the specific language governing 
permissions and limitations under
    + * the License.
    + */
    +package org.apache.geode.session.tests;
    +
    +import static org.junit.Assert.assertEquals;
    +
    +import java.io.IOException;
    +import java.net.URISyntaxException;
    +import java.util.concurrent.TimeUnit;
    +
    +import javax.servlet.http.HttpSession;
    +
    +import org.awaitility.Awaitility;
    +import org.junit.BeforeClass;
    +import org.junit.Test;
    +
    +import org.apache.geode.cache.Cache;
    +import org.apache.geode.cache.Region;
    +import org.apache.geode.modules.session.functions.GetSessionCount;
    +import org.apache.geode.test.dunit.DUnitEnv;
    +
    +/**
    + * Jetty 9 Client Server tests
    + *
    + * Runs all the tests in {@link CargoTestBase} on the Jetty 9 install, 
setup in the
    + * {@link #setupJettyInstall()} method before tests are run.
    + */
    +public class Jetty9CachingClientServerTest extends 
GenericAppServerClientServerTest {
    +  private static ContainerInstall install;
    +
    +  @BeforeClass
    +  public static void setupJettyInstall() throws Exception {
    +    install = new 
GenericAppServerInstall(GenericAppServerInstall.GenericAppServerVersion.JETTY9,
    +        ContainerInstall.ConnectionType.CACHING_CLIENT_SERVER,
    +        ContainerInstall.DEFAULT_INSTALL_DIR + "Jetty9ClientServerTest");
    --- End diff --
    
    Change this install directory?


> Generic session replication module leaves native sessions around even in 
> proxy mode
> -----------------------------------------------------------------------------------
>
>                 Key: GEODE-3513
>                 URL: https://issues.apache.org/jira/browse/GEODE-3513
>             Project: Geode
>          Issue Type: Bug
>          Components: http session
>            Reporter: Dan Smith
>            Assignee: Dan Smith
>
> The session module for app servers is creating native sessions in the user's 
> application server and never cleaning them up in client/server mode.
> In proxy mode this means we end up using up memory for the session, even 
> though we are fetching the session from the server on every request.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to