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

ASF subversion and git services commented on GEODE-6867:
--------------------------------------------------------

Commit c058bb2b02e037ebc747b5a2909a31afb0ef153c in geode's branch 
refs/heads/develop from Ryan McMahon
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=c058bb2 ]

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

This reverts commit b31d9f335ff202ba795bcf0567abcc032a39f979.


> Fix documentation for session state caching
> -------------------------------------------
>
>                 Key: GEODE-6867
>                 URL: https://issues.apache.org/jira/browse/GEODE-6867
>             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|https://geode.apache.org/docs/guide/19/tools_modules/http_session_mgmt/tomcat_installing_the_module.html]
>  for Tomcat and [setting up 
> section|https://geode.apache.org/docs/guide/19/tools_modules/http_session_mgmt/weblogic_setting_up_the_module.html]
>  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 TomcatInstall.java:
> {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
(v8.3.2#803003)

Reply via email to