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:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to