geode-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (Jira)" <>
Subject [jira] [Commented] (GEODE-6867) Fix documentation for session state caching
Date Tue, 10 Sep 2019 16:07:00 GMT


ASF subversion and git services commented on GEODE-6867:

Commit 00d81d924b3c4e405434f3d13a1df1788841b7a1 in geode's branch refs/heads/feature/GEODE-7168
from Bruce Schuchardt
[;h=00d81d9 ]

Revert "Revert "GEODE-6867: Updating Tomcat Session State Module docs (#3886)""

This reverts commit c058bb2b02e037ebc747b5a2909a31afb0ef153c.

> Fix documentation for session state caching
> -------------------------------------------
>                 Key: GEODE-6867
>                 URL:
>             Project: Geode
>          Issue Type: Bug
>          Components: docs, http session
>            Reporter: Ryan McMahon
>            Priority: Major
>              Labels: GeodeCommons
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
> This story outlines two major issues in the session state docs, but I think the docs
deserve a fairly comprehensive review to determine if there is any other stale/incorrect information. 
The two issues are:
>  # Required jar list is missing some required jars
>  # Need to indicate that a locator MUST be available for peer-to-peer topology
> More details on both issues below:
> We should review the session state documentation for completeness/correctness.  When
attempting to install and configure the Tomcat and AppServer session modules by following
the docs, I found that there were missing jars in the [installation section|] for
Tomcat and [setting up section|] for
AppServer.  Namely, I had to add these missing jars (as of 9.8, maybe earlier).
>  * micrometer-core jar
>  * geode-commons jar
>  * geode-management jar
> The first is a new jar added since this documentation was written, and I'm assuming the
code was restructured to require the other two jars as dependencies.  I'm not sure how we
can ensure that this list is up to date.  Some of our system level tests for session state
have to include the same list.  From
> {code:java}
> private static final String[] tomcatRequiredJars =
>  {"antlr", "commons-io", "commons-lang", "commons-validator", "fastutil", "geode-common",
>  "geode-core", "geode-management", "javax.transaction-api", "jgroups", "log4j-api",
>  "log4j-core", "log4j-jul", "micrometer", "shiro-core", "jetty-server", "jetty-util",
>  "jetty-http", "jetty-io"};{code}
> And you can see that this list was updated to make the tests pass when dependencies changed.
> The other major issue I found while running through the steps is that you MUST start
a locator in the peer-to-peer topology for session state, because multicast UDP discovery
is not available/supported in Geode.

This message was sent by Atlassian Jira

View raw message