https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #19 from Mark Thomas ---
(In reply to Huxing Zhang from comment #18)
> Hi Mark,
>
> Glad to hear the patch has been accepted! Thanks for your hard work!
> BTW, I am interested in becoming a tomcat committer, any advice on that?
Ke
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #18 from Huxing Zhang ---
Hi Mark,
Glad to hear the patch has been accepted! Thanks for your hard work!
BTW, I am interested in becoming a tomcat committer, any advice on that?
--
You are receiving this mail because:
You are the
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
Mark Thomas changed:
What|Removed |Added
Resolution|--- |FIXED
Status|REOPENED
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #16 from Mark Thomas ---
I opted for the simpler replacement of StandardClassLoader with URLClassLoader
wherever the StandardClassLoader was used.
--
You are receiving this mail because:
You are the assignee for the bug.
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #15 from Huxing Zhang ---
To make StandardClassLoader parallel capable, we can simply add JRECompact to
the Bootstrap class loader's class path, and then apply the static
initialization of WebappClassLoaderBase's to it.
If we don't
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #14 from Mark Thomas ---
Removing deprecated code is not an option since it may break existing code.
That is why it was deprecated rather than deleted in the first place. The patch
needs to be re-worked not to do this.
I've started
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #13 from Huxing Zhang ---
Is there anything else I can do to push it forward?
The patch respects the discussion in bug 56530.
--
You are receiving this mail because:
You are the assignee for the bug.
-
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #12 from Huxing Zhang ---
Hi, Any update on this?
--
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubsc
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
Huxing Zhang changed:
What|Removed |Added
Resolution|DUPLICATE |---
Status|RESOLVED
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
Huxing Zhang changed:
What|Removed |Added
CC||huxing.zh...@gmail.com
--- Comment #11
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #10 from Mark Thomas ---
(In reply to Alex Koturanov from comment #8)
> Comment on attachment 32554 [details]
> Patch to work with JDK1.6
Doesn't implement parallel loading.
Creates significant risk of a ConcurrentModificationExce
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #9 from Alex Koturanov ---
Created attachment 32584
--> https://bz.apache.org/bugzilla/attachment.cgi?id=32584&action=edit
Lock free read for already loaded/cached classes
--
You are receiving this mail because:
You are the assi
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
Alex Koturanov changed:
What|Removed |Added
Attachment #32554|0 |1
is obsolete|
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #8 from Alex Koturanov ---
Comment on attachment 32554
--> https://bz.apache.org/bugzilla/attachment.cgi?id=32554
Patch to work with JDK1.6
--- WebappClassLoader_r1661811.java2015-03-09 21:53:27.0 +
+++ WebappClas
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #7 from Mark Thomas ---
This patch is also wrong.
THe correct way to address this is described in bug 56530.
--
You are receiving this mail because:
You are the assignee for the bug.
-
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
Alex Koturanov changed:
What|Removed |Added
Attachment #32553|0 |1
is obsolete|
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
Alex Koturanov changed:
What|Removed |Added
Attachment #32554|Patch to work with JDK6 |Patch to work with JDK1.6
desc
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #6 from Alex Koturanov ---
Attachment 'Parallel classloading port from Tc8.0 into Tc7.0' was incomplete
and should not be used, apologies for attaching a wrong diff.
'Patch to work with JDK6' re-implements getClassLoadingLock() in
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #5 from Alex Koturanov ---
Created attachment 32554
--> https://bz.apache.org/bugzilla/attachment.cgi?id=32554&action=edit
Patch to work with JDK6
--
You are receiving this mail because:
You are the assignee for the bug.
--
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #4 from Mark Thomas ---
Because we don't have separate Bugzilla entries for exactly the same issue in
different versions of Tomcat.
--
You are receiving this mail because:
You are the assignee for the bug.
---
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #3 from Alex Koturanov ---
The fix in bug 56530 is for Tomcat 8, not Tomcat 7. Why my request is marked as
resolved?
--
You are receiving this mail because:
You are the assignee for the bug.
--
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
--- Comment #2 from Mark Thomas ---
And for the record the patch is nowhere close to correct.
--
You are receiving this mail because:
You are the assignee for the bug.
-
To
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
Mark Thomas changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bz.apache.org/bugzilla/show_bug.cgi?id=57681
Alex Koturanov changed:
What|Removed |Added
CC||kotur...@amazon.com
O
24 matches
Mail list logo