Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed in Intellij 13 #13

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fixed in Intellij 13 #13

wants to merge 2 commits into from

Conversation

peterox
Copy link

@peterox peterox commented Mar 28, 2014

Added better connection pooling to RallyRestApi.

@crc83
Copy link
Owner

crc83 commented Jun 28, 2014

I'm really doubt if we need this at all. This connection is not used so often and amount of data is not so big.

@peterox
Copy link
Author

peterox commented Jul 4, 2014

This was needed so that a different connection is used by the server config dialog and any background tasks that use the repository. Without this fix the single connection created is cleaned up and the background tasks fail.

@crc83
Copy link
Owner

crc83 commented Oct 20, 2014

Too much file changed. Could you please pull only those related to an issue?

@cweagans
Copy link

cweagans commented Nov 5, 2015

FWIW, I started rerolling this for IntelliJ 15 today, and it causes a ton of files to change. Probably pretty unavoidable.

The connection pooling stuff could be it's own PR, though.

@crc83
Copy link
Owner

crc83 commented Jan 9, 2019

Sorry for ages of silence. I'll integrate these changes, but as a commit from my repo).
Peter, I'll mention you in class header.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants