Skip to content
This repository has been archived by the owner on Oct 12, 2023. It is now read-only.

Feature/add cacheSize config, fix tomcat log cache warning. #171

Closed

Conversation

xingxiuyi
Copy link

some jar like vaadin, contains java source file and other resources file. Tomcat will log warning for each file.

solve this problem: add cacheSize config.

FIX: #170

@xingxiuyi xingxiuyi changed the title add cacheSize config, fix tomcat log cache warning. Feature/add cacheSize config, fix tomcat log cache warning. Jan 8, 2018
@xingxiuyi
Copy link
Author

only support tomcat 8+

use cacheSize property, tomcat 6 & tomcat 7 will print a warning.

Copy link
Owner

@bmuschko bmuschko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the pull request!

Could you please add integration tests for Tomcat < 8.x and >= 8.x for verify the correct behavior?

We'll also want to add the new property to the documentation in README.md.

@xingxiuyi
Copy link
Author

OK

please wait a few days.

@bmuschko
Copy link
Owner

bmuschko commented Mar 2, 2018

@xingxiuyi Are you still interested in finishing off this pull request?

@xingxiuyi
Copy link
Author

@bmuschko I'm sorry, my work was too busy before. I will finish the pull_request as soon as possible.

@bmuschko
Copy link
Owner

@xingxiuyi Is this still happening? Otherwise I am going to close this PR for now.

@bmuschko
Copy link
Owner

Closing this pull request as new test has been provided. Please let me know if you'd like to continue adding one in which case we can reopen the pull request.

@bmuschko bmuschko closed this May 11, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants