I'll assume DynamicMBeanProxy will stay for now and make sure the jtc/util projects in Gump still show a dependency on JMX.
Larry > -----Original Message----- > From: Costin Manolache [mailto:[EMAIL PROTECTED]] > Sent: Monday, January 27, 2003 3:55 PM > To: [EMAIL PROTECTED] > Subject: RE: cvs commit: > jakarta-tomcat-connectors/util/java/org/apache/tomcat/util/thr > eads ThreadPool.java ThreadPoolMX.java > > > Larry Isaacs wrote: > > > Hi Costin, > > > > jtc/util still contains org.apache.tomcat.util.mx.DynamicMBeanProxy > > which imports "javax.management.*". Is the intent to > update or remove > > this class so the dependency on JMX can be removed? Thanks. > > To deprecate - for sure, there is no point in maintaining it when the > same functionality is in modeler. I made quite a few improvements to > the equivalent code in modeler ( support for more types, > automatic detection > of .xml descriptors, etc ). > > About remove - I don't know if it was included in any > official release, > and if removing it won't break code. I know of a use in 3.3, is anyone > else using it ? > > We definitely need a way to document what APIs are "FROZEN" > ( like mozilla ). > > > Costin > > > > > Cheers, > > Larry > > > >> -----Original Message----- > >> From: Costin Manolache [mailto:[EMAIL PROTECTED]] > >> Sent: Monday, January 27, 2003 10:39 AM > >> To: [EMAIL PROTECTED] > >> Subject: Re: cvs commit: > >> jakarta-tomcat-connectors/util/java/org/apache/tomcat/util/thr > >> eads ThreadPool.java ThreadPoolMX.java > >> > >> > >> FYI: I did the same in the main branch. All of DynamicMBean > >> features are > >> now porte to modeler. There is no need for JMX dependency > in jtc/util > >> ( the branch is still needed in coyote, jk, http11 ) > >> > >> Costin > >> > >> > >> > >> [EMAIL PROTECTED] wrote: > >> > billbarker 2003/01/24 20:49:24 > >> > > >> > Modified: util/java/org/apache/tomcat/util/threads > >> Tag: coyote_10 > >> > ThreadPool.java > >> > Removed: util/java/org/apache/tomcat/util/threads > >> Tag: coyote_10 > >> > ThreadPoolMX.java > >> > Log: > >> > Removing JMX dependencies from this branch (as voted on > >> in tomcat-dev). > >> > > >> > Revision Changes Path > >> > No revision > >> > > >> > > >> > No revision > >> > > >> > > >> > 1.9.2.1 +0 -9 > >> jakarta-tomcat-connectors/util/java/org/apache/tomcat/util/thr > >> eads/ThreadPool.java > >> > > >> > Index: ThreadPool.java > >> > > >> =================================================================== > >> > RCS file: > >> /home/cvs/jakarta-tomcat-connectors/util/java/org/apache/tomca > >> t/util/threads/ThreadPool.java,v > >> > retrieving revision 1.9 > >> > retrieving revision 1.9.2.1 > >> > diff -u -r1.9 -r1.9.2.1 > >> > --- ThreadPool.java 14 Jan 2003 18:47:39 -0000 1.9 > >> > +++ ThreadPool.java 25 Jan 2003 04:49:24 -0000 > 1.9.2.1 > >> > @@ -159,15 +159,6 @@ > >> > * call register() in order to set a name. > >> > */ > >> > public static ThreadPool createThreadPool(boolean jmx) { > >> > - if( jmx ) { > >> > - try { > >> > - Class.forName( > >> "org.apache.commons.modeler.Registry"); > >> > - Class tpc=Class.forName( > >> "org.apache.tomcat.util.threads.ThreadPoolMX"); > >> > - ThreadPool res=(ThreadPool)tpc.newInstance(); > >> > - return res; > >> > - } catch( Exception ex ) { > >> > - } > >> > - } > >> > return new ThreadPool(); > >> > } > >> > >> > >> > >> > >> -- > >> To unsubscribe, e-mail: > >> <mailto:tomcat-dev-> [EMAIL PROTECTED]> > >> For > >> additional commands, > >> e-mail: <mailto:[EMAIL PROTECTED]> > >> > >> > > > > -- > To unsubscribe, e-mail: > <mailto:tomcat-dev-> [EMAIL PROTECTED]> > For > additional commands, > e-mail: <mailto:[EMAIL PROTECTED]> > > -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
