[JBoss-dev] CVS update: jbosscx/src/main/org/jboss/resource/connectionmanager ManagedConnectionPoolFactory.java

2001-11-21 Thread Doug Ferguson

  User: thedug  
  Date: 01/11/21 10:14:51

  Modified:src/main/org/jboss/resource/connectionmanager
ManagedConnectionPoolFactory.java
  Log:
  destroy() should actually close the connection so that the tcp connection isn't
  left open.
  
  Revision  ChangesPath
  1.3   +16 -0 
jbosscx/src/main/org/jboss/resource/connectionmanager/ManagedConnectionPoolFactory.java
  
  Index: ManagedConnectionPoolFactory.java
  ===
  RCS file: 
/cvsroot/jboss/jbosscx/src/main/org/jboss/resource/connectionmanager/ManagedConnectionPoolFactory.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- ManagedConnectionPoolFactory.java 2001/09/11 18:38:58 1.2
  +++ ManagedConnectionPoolFactory.java 2001/11/21 18:14:51 1.3
  @@ -63,6 +63,22 @@
 pool = null;
  }
   
  +/**
  + * Permanently closes an object, after it is removed from the  pool.  The
  + * object will not be returned to the pool - after this, it is  gone.  This
  + * is called when the pool shrinks, and when the pool is shut  down.
  + */
  +public void deleteObject(Object pooledObject) {
  +try{
  +((ManagedConnection)pooledObject).destroy();
  +}catch(ResourceException e) {
  +log.println(Unable to destroy ManagedConnection: +e);
  +}
  +
  + }
  +
  +
  +
  /**
   * Creates a new ManagedConnection with the specified parameters.
   *
  
  
  

___
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development



[JBoss-dev] CVS update: jbosscx/src/main/org/jboss/resource/connectionmanager ManagedConnectionPoolFactory.java

2001-11-21 Thread Scott M Stark

  User: starksm 
  Date: 01/11/21 11:41:16

  Modified:src/main/org/jboss/resource/connectionmanager
ManagedConnectionPoolFactory.java
  Log:
  Fix the bad check-in by thedug
  
  Revision  ChangesPath
  1.4   +9 -5  
jbosscx/src/main/org/jboss/resource/connectionmanager/ManagedConnectionPoolFactory.java
  
  Index: ManagedConnectionPoolFactory.java
  ===
  RCS file: 
/cvsroot/jboss/jbosscx/src/main/org/jboss/resource/connectionmanager/ManagedConnectionPoolFactory.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- ManagedConnectionPoolFactory.java 2001/11/21 18:14:51 1.3
  +++ ManagedConnectionPoolFactory.java 2001/11/21 19:41:16 1.4
  @@ -68,14 +68,18 @@
* object will not be returned to the pool - after this, it is  gone.  This
* is called when the pool shrinks, and when the pool is shut  down.
*/
  -public void deleteObject(Object pooledObject) {
  -try{
  +public void deleteObject(Object pooledObject)
  +{
  +try
  +{
   ((ManagedConnection)pooledObject).destroy();
  -}catch(ResourceException e) {
  -log.println(Unable to destroy ManagedConnection: +e);
   }
  +catch(ResourceException e)
  +{
  +log.error(Unable to destroy ManagedConnection: , e);
  +}
   
  - }
  +   }
   
   
   
  
  
  

___
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development



RE: [JBoss-dev] CVS update: jbosscx/src/main/org/jboss/resource/connectionmanager ManagedConnectionPoolFactory.java

2001-11-21 Thread Ferguson, Doug

Sorry,

I was merging a change I made to 2.4 and didn't realize the log method had
changed.

Also, I noticed that you changed formatting. Does JBoss enforce coding
standards in
regards to formatting? If so,  where can I find this information?

Thanks,
d.

-Original Message-
From: Scott M Stark [mailto:[EMAIL PROTECTED]]
Sent: Wednesday, November 21, 2001 1:41 PM
To: [EMAIL PROTECTED]
Subject: [JBoss-dev] CVS update:
jbosscx/src/main/org/jboss/resource/connectionmanager
ManagedConnectionPoolFactory.java


  User: starksm 
  Date: 01/11/21 11:41:16

  Modified:src/main/org/jboss/resource/connectionmanager
ManagedConnectionPoolFactory.java
  Log:
  Fix the bad check-in by thedug
  
  Revision  ChangesPath
  1.4   +9 -5
jbosscx/src/main/org/jboss/resource/connectionmanager/ManagedConnectionPoolF
actory.java
  
  Index: ManagedConnectionPoolFactory.java
  ===
  RCS file:
/cvsroot/jboss/jbosscx/src/main/org/jboss/resource/connectionmanager/Managed
ConnectionPoolFactory.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- ManagedConnectionPoolFactory.java 2001/11/21 18:14:51 1.3
  +++ ManagedConnectionPoolFactory.java 2001/11/21 19:41:16 1.4
  @@ -68,14 +68,18 @@
* object will not be returned to the pool - after this, it is  gone.
This
* is called when the pool shrinks, and when the pool is shut  down.
*/
  -public void deleteObject(Object pooledObject) {
  -try{
  +public void deleteObject(Object pooledObject)
  +{
  +try
  +{
   ((ManagedConnection)pooledObject).destroy();
  -}catch(ResourceException e) {
  -log.println(Unable to destroy ManagedConnection: +e);
   }
  +catch(ResourceException e)
  +{
  +log.error(Unable to destroy ManagedConnection: , e);
  +}
   
  - }
  +   }
   
   
   
  
  
  

___
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development

___
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development



Re: [JBoss-dev] CVS update: jbosscx/src/main/org/jboss/resource/connectionmanager ManagedConnectionPoolFactory.java

2001-11-21 Thread Scott M Stark

Standards are here:
http://www.jboss.org/developers/guide/codestyle.jsp


Scott Stark
Chief Technology Officer
JBoss Group, LLC

- Original Message -
From: Ferguson, Doug [EMAIL PROTECTED]
To: 'Scott M Stark' [EMAIL PROTECTED];
[EMAIL PROTECTED]
Sent: Wednesday, November 21, 2001 12:15 PM
Subject: RE: [JBoss-dev] CVS update:
jbosscx/src/main/org/jboss/resource/connectionmanager
ManagedConnectionPoolFactory.java


 Sorry,

 I was merging a change I made to 2.4 and didn't realize the log method had
 changed.

 Also, I noticed that you changed formatting. Does JBoss enforce coding
 standards in
 regards to formatting? If so,  where can I find this information?

 Thanks,
 d.

 -Original Message-
 From: Scott M Stark [mailto:[EMAIL PROTECTED]]
 Sent: Wednesday, November 21, 2001 1:41 PM
 To: [EMAIL PROTECTED]
 Subject: [JBoss-dev] CVS update:
 jbosscx/src/main/org/jboss/resource/connectionmanager
 ManagedConnectionPoolFactory.java


   User: starksm
   Date: 01/11/21 11:41:16

   Modified:src/main/org/jboss/resource/connectionmanager
 ManagedConnectionPoolFactory.java
   Log:
   Fix the bad check-in by thedug

   Revision  ChangesPath
   1.4   +9 -5

jbosscx/src/main/org/jboss/resource/connectionmanager/ManagedConnectionPoolF
 actory.java

   Index: ManagedConnectionPoolFactory.java
   ===
   RCS file:

/cvsroot/jboss/jbosscx/src/main/org/jboss/resource/connectionmanager/Managed
 ConnectionPoolFactory.java,v
   retrieving revision 1.3
   retrieving revision 1.4
   diff -u -r1.3 -r1.4
   --- ManagedConnectionPoolFactory.java 2001/11/21 18:14:51 1.3
   +++ ManagedConnectionPoolFactory.java 2001/11/21 19:41:16 1.4
   @@ -68,14 +68,18 @@
 * object will not be returned to the pool - after this, it is
gone.
 This
 * is called when the pool shrinks, and when the pool is shut
down.
 */
   -public void deleteObject(Object pooledObject) {
   -try{
   +public void deleteObject(Object pooledObject)
   +{
   +try
   +{
((ManagedConnection)pooledObject).destroy();
   -}catch(ResourceException e) {
   -log.println(Unable to destroy ManagedConnection: +e);
}
   +catch(ResourceException e)
   +{
   +log.error(Unable to destroy ManagedConnection: , e);
   +}

   - }
   +   }







 ___
 Jboss-development mailing list
 [EMAIL PROTECTED]
 https://lists.sourceforge.net/lists/listinfo/jboss-development



___
Jboss-development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development