Archiva

Support database connection pooling

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.1
  • Component/s: system
  • Labels:
    None
  • Number of attachments :
    0

Issue Links

Activity

Hide
Wendy Smoak added a comment -

Deng, can you add a description to the issue? I looked for any discussion to summarize, and didn't see anything. I'm sure it's straightforward, just seems odd to see a new feature in progress with no information about it.

Show
Wendy Smoak added a comment - Deng, can you add a description to the issue? I looked for any discussion to summarize, and didn't see anything. I'm sure it's straightforward, just seems odd to see a new feature in progress with no information about it.
Hide
Maria Odea Ching added a comment -

I think this was mentioned during the migration of the plexus runtime to jetty in IRC. It's basically just allowing db connection pooling support in the jetty bundle.

Here's something from the IRC logs..
Mar 27 10:54:19 <deng> brett, i think the derby.system.home property is putting the database dir inside the logs
Mar 27 10:54:51 <brett> hehe
Mar 27 10:54:56 <brett> oops
Mar 27 10:55:11 <deng> hehe
Mar 27 10:55:24 <brett> make it ../data/databases then?
Mar 27 10:55:40 <deng> ok
Mar 27 10:56:40 <deng> thanks
Mar 27 11:03:56 <brett> deng: while you are looking at the databases, did I mention connection pooling?
Mar 27 11:07:29 <deng> brett, not yet
Mar 27 11:07:41 <deng> should we configure that by default in the jetty.xml?
Mar 27 11:08:11 <deng> or just add a how-to to the docs?
Mar 27 11:08:44 <brett> maybe it can be a new JIRA... I just thought we had it before
Mar 27 11:08:58 <brett> but I don't think we did
Mar 27 11:09:30 <deng> yeah, i didn't see it before
Mar 27 11:09:52 <brett> I think commons-dbcp sits behind the naming stuff I was using
Mar 27 11:09:58 <brett> but I'm not sure if it pools by default
Mar 27 11:10:01 <brett> no matter

Show
Maria Odea Ching added a comment - I think this was mentioned during the migration of the plexus runtime to jetty in IRC. It's basically just allowing db connection pooling support in the jetty bundle. Here's something from the IRC logs.. Mar 27 10:54:19 <deng> brett, i think the derby.system.home property is putting the database dir inside the logs Mar 27 10:54:51 <brett> hehe Mar 27 10:54:56 <brett> oops Mar 27 10:55:11 <deng> hehe Mar 27 10:55:24 <brett> make it ../data/databases then? Mar 27 10:55:40 <deng> ok Mar 27 10:56:40 <deng> thanks Mar 27 11:03:56 <brett> deng: while you are looking at the databases, did I mention connection pooling? Mar 27 11:07:29 <deng> brett, not yet Mar 27 11:07:41 <deng> should we configure that by default in the jetty.xml? Mar 27 11:08:11 <deng> or just add a how-to to the docs? Mar 27 11:08:44 <brett> maybe it can be a new JIRA... I just thought we had it before Mar 27 11:08:58 <brett> but I don't think we did Mar 27 11:09:30 <deng> yeah, i didn't see it before Mar 27 11:09:52 <brett> I think commons-dbcp sits behind the naming stuff I was using Mar 27 11:09:58 <brett> but I'm not sure if it pools by default Mar 27 11:10:01 <brett> no matter
Hide
Maria Odea Ching added a comment -

Fixed in trunk -r669774:

  • configure jetty to use commons-dbcp for db connection pool
  • added commons-dbcp 1.2.1 as dependency
Show
Maria Odea Ching added a comment - Fixed in trunk -r669774:
  • configure jetty to use commons-dbcp for db connection pool
  • added commons-dbcp 1.2.1 as dependency

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: