Hi, Following on this, as I have raised a PR https://github.com/docker-library/tomcat/pull/202 to add the SCC options into the OpenJ9 Docker file to avail this improvement, I got comments saying that they would do only if upstream recommends. As all individual users may not know that this option helps, we would like to get this added into OpenJ9 Docker file itself. I wanted to check with the community, if you are OK adding it ? Or any suggestions you would like to share on this ?
Thank You! ```If this is something upstream recommends, shouldn't it be noted in their documentation? Maybe a configuration file/option which tells catalina.sh to use their preferred options? It seems like even a dedicated subcommand (like run) to generate them would be useful, especially if the options during generation need to be different than the options during runtime. Given that I'm not aware of any documentation from upstream which recommends this and that this is something users can accomplish in a dependent Dockerfile that's FROM tomcat:...-openj9, I'm inclined to not do this at this level for now.``` On Thu, May 21, 2020 at 2:02 PM Martin Grigorov <mgrigo...@apache.org> wrote: > > > On Mon, May 18, 2020 at 2:45 PM kusuma chalasani <kusuma...@gmail.com> > wrote: > >> Hi all, >> >> I'm a Performance Analyst for JVM runtime. I recently tried out some >> experiments to compare Tomcat startup performance in Docker using >> AdoptOpenJDK-OpenJ9 >> >> I found I was able to get ~30% faster startup time with very minimal >> change in footprint when I enable SharedClass Cache when I compared to the >> default(which is No Shareclasses). I wrote a detailed blog post here >> https://blog.openj9.org/2020/05/18/startup-performance-of-tomcat-in-docker/ >> where >> I outlined what and how I measured. >> >> I also did raise a PR for the same - >> https://github.com/docker-library/tomcat/pull/202 >> >> We figured we'd reach out to you guys so you can take a look, and maybe >> try it out and let us know if you have any comments/questions. >> > > Thank you for sharing your findings with us, Kusuma! > > >> >> >> Thanks, >> Kusuma. >> > -- Thanks, Kusuma.