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

[ci fw-only Java/servlet] Use Jdbc3ConnectionPool from PostgreJDBC #3365

Closed
wants to merge 1 commit into from

Conversation

zloster
Copy link
Contributor

@zloster zloster commented Mar 6, 2018

According to the Resin documentation the Driver class should be a last resort.

@NateBrady23
Copy link
Member

@zloster same with this one if you don't mind targeting the docker branch. I just don't want to miss it as we're probably going to do some cherry-picking from master.

@zloster
Copy link
Contributor Author

zloster commented Mar 12, 2018

@nbrady-techempower Java/servlet is not updated in the Docker branch with the changes from #3324. Also it is not migrated to the Docker infrastructure yet.

@NateBrady23
Copy link
Member

@zloster You are right! I took a peek at the travis file and thought they were all set.

@zloster
Copy link
Contributor Author

zloster commented Mar 12, 2018

@nbrady-techempower Is the "PR: Please Update" label still applicable here?

@NateBrady23
Copy link
Member

This is done in #3439 now, thanks!

@zloster zloster deleted the issue-43 branch October 11, 2018 13:15
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.

None yet

2 participants