kamnani commented on pull request #351: URL: https://github.com/apache/tomcat/pull/351#issuecomment-694967799
The primary aim of this PR is to trim off the white spaces. I can give you an example here. Optimized ``` | <html> -- | -- | <head> | <title>Quick Servlet Demo</title> | </head> | <body> | <b>Tomcat 9 Test app</b> | <p>This has over 1000 Jar Files</p> | <p><b>FastStartup Flag: true</b></p> | <p><b>OptimizeJSPs Flag: true</b></p> | <p>To see server Logs: <a href="/logs/catalina.out">Click Here</a></p> | <p>To see Optimized Render times of tags: <a href="/tomcat9Testing">Click Here</a></p> | </body> | </html> ``` Unoptimized: ``` <html> -- | <head> | <title>Quick Servlet Demo</title> | </head> | <body> | <b>Tomcat 9 Test app</b> | <p>This has over 1000 Jar Files</p> | | <p><b>FastStartup Flag: false</b></p> | <p><b>OptimizeJSPs Flag: false</b></p> | <p>To see server Logs: <a href="/logs/catalina.out">Click Here</a></p> | <p>To see Optimized Render times of tags: <a href="/tomcat9Testing">Click Here</a></p> | | | | | | | </body> | </html> ``` For large JSP files these white spaces are tremendous and also useless when passed over the network. The white spaces are good for readability for the user, and are not necessarily needed as source-compiled files. Removing these white spaces result in fewer writes to internal buffers and fewer flushes to the network and thus results in lower latency. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
