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

Benjamin P Ross commented on GEODE-6867:
----------------------------------------

We identified that there are several issues in the docs when setting up the 
[client/server 
topology|https://geode.apache.org/docs/guide/19/tools_modules/http_session_mgmt/tomcat_setting_up_the_module.html]
 with Tomcat as well.

This entire section of the document could be included as a script along with 
our module jars, instead of requiring the user to do manual steps.

If we just update the manual steps, here are some issues we found:
1. The list of jars to include in the CLASSPATH is incomplete.  It either needs 
to be updated to include the missing jars, or perhaps a wildcard on the libs 
directory would work.  It seems like the TC_VER and INSTANCE variables aren't 
necessary, and instead $CATALINA_HOME/lib can be used.
2. Tomcat 6.0 is deprecated and the module is no longer included in our 
packages, so we should remove the section in the docs on that version.
3. Copy pasting doesn't work for config changes because of extra whitespace in 
the doc

> 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
>
> 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://gemfire.docs.pivotal.io/98/geode/tools_modules/http_session_mgmt/tomcat_installing_the_module.html]]
>  for Tomcat and [setting up 
> section|[https://gemfire.docs.pivotal.io/98/geode/tools_modules/http_session_mgmt/weblogic_setting_up_the_module.html]].
>   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
(v7.6.3#76005)

Reply via email to