Skip to content
This repository has been archived by the owner on Jan 3, 2019. It is now read-only.

Connection problem (partial) fix #80

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hading
Copy link

@hading hading commented Mar 20, 2015

Make limits on maximum number of connections in pool.
Release connections after they are used.

This is regarding #79.

It's not fully complete. The limits on the connection pool are currently just magic numbers; I assume you'd want to work that into configuration in whatever standard way you have of doing it.

The tests pass at this point, and the connections do not pile up.

I do, however (as noted in the JIRA ticket) now experience some items not being indexed. I don't know if this is because of problems in my code or if messages are being dropped somehow. I don't tend to see it until I start indexing around 5000-10000 things or so all at once.

I don't know if I'll look further - it's been suggested that we look into using the Camel integration instead. But since I already had this might as well send it along.

Release connections after they are used.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant