-
Notifications
You must be signed in to change notification settings - Fork 1.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Startup XmlConfiguration WARN on Arg threadpool #4631
Labels
Bug
For general bugs on Jetty side
Comments
This is ultimately a bad warning, the argument isn't being ignored, and is being used properly. It's just a later check for |
joakime
added a commit
that referenced
this issue
Mar 2, 2020
Signed-off-by: Joakim Erdfelt <[email protected]>
joakime
added a commit
that referenced
this issue
Mar 2, 2020
… for <Configure> Signed-off-by: Joakim Erdfelt <[email protected]>
joakime
added a commit
that referenced
this issue
Mar 2, 2020
Signed-off-by: Joakim Erdfelt <[email protected]>
joakime
added a commit
that referenced
this issue
Mar 3, 2020
Signed-off-by: Joakim Erdfelt <[email protected]>
joakime
added a commit
that referenced
this issue
Mar 3, 2020
Signed-off-by: Joakim Erdfelt <[email protected]>
joakime
added a commit
that referenced
this issue
Mar 3, 2020
Signed-off-by: Joakim Erdfelt <[email protected]>
joakime
added a commit
that referenced
this issue
Mar 3, 2020
+ new testcase where <Arg> is needed, but is not the first node Signed-off-by: Joakim Erdfelt <[email protected]>
gregw
added a commit
that referenced
this issue
Mar 10, 2020
… for <Configure> (#4632) * Issue #4631 - Fixing XML comment that was accidentally reformatted Signed-off-by: Joakim Erdfelt <[email protected]> * Issue #4631 - Warning about skipping of <Arg> nodes is in wrong place for <Configure> Signed-off-by: Joakim Erdfelt <[email protected]> * Issue #4631 - Improving testcase Signed-off-by: Joakim Erdfelt <[email protected]> * Issue #4631 - Removing test classes Signed-off-by: Joakim Erdfelt <[email protected]> * Issue #4631 - Cleaning up configure with index per PR review Signed-off-by: Joakim Erdfelt <[email protected]> * Issue #4631 - More named arg test cases Signed-off-by: Joakim Erdfelt <[email protected]> * Issue #4631 - Add testConfiguredWithNamedArgNotFirst + new testcase where <Arg> is needed, but is not the first node Signed-off-by: Joakim Erdfelt <[email protected]> * Cleanup configuration index usage Signed-off-by: Greg Wilkins <[email protected]> Co-authored-by: Greg Wilkins <[email protected]>
Fixed by #4632. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Originally posted by @TezzerFX in #4620 (comment)
There's a WARN message on startup about the threadpool ...
The text was updated successfully, but these errors were encountered: