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

Upgrade postgresql to 12 #3019

Merged
merged 2 commits into from
Sep 28, 2020
Merged

Upgrade postgresql to 12 #3019

merged 2 commits into from
Sep 28, 2020

Conversation

RemiDesgrange
Copy link
Contributor

I did not try the code yet (dark story about capted internet) I'll try after the 9th

@landryb
Copy link
Member

landryb commented Apr 1, 2020

fwiw im using psql 12/pgis 3 in some production servers (not georchestra yet, but nextcloud/mapserver/drupal/gdal clients) and it works pretty fine, the only drawback being that you need a psql 12 client for some operations (ie at least \d)

@fvanderbiest
Copy link
Member

LGTM

@fvanderbiest
Copy link
Member

At runtime it's a different story ...

proxy_1                     | log4j:ERROR Failed to insert the log
proxy_1                     | java.sql.SQLException: An attempt by a client to checkout a Connection has timed out.
proxy_1                     | 	at com.mchange.v2.sql.SqlUtils.toSQLException(SqlUtils.java:118)
proxy_1                     | 	at com.mchange.v2.sql.SqlUtils.toSQLException(SqlUtils.java:77)
proxy_1                     | 	at com.mchange.v2.c3p0.impl.C3P0PooledConnectionPool.checkoutPooledConnection(C3P0PooledConnectionPool.java:690)
proxy_1                     | 	at com.mchange.v2.c3p0.impl.AbstractPoolBackedDataSource.getConnection(AbstractPoolBackedDataSource.java:140)
proxy_1                     | 	at org.georchestra.ogcservstatistics.dataservices.DataServicesConfiguration.getConnection(DataServicesConfiguration.java:73)
proxy_1                     | 	at org.georchestra.ogcservstatistics.log4j.OGCServicesAppender.insert(OGCServicesAppender.java:205)
proxy_1                     | 	at org.georchestra.ogcservstatistics.log4j.OGCServicesAppender.lambda$append$0(OGCServicesAppender.java:196)
proxy_1                     | 	at java.util.concurrent.CompletableFuture$AsyncRun.run(CompletableFuture.java:1640)
proxy_1                     | 	at java.util.concurrent.CompletableFuture$AsyncRun.exec(CompletableFuture.java:1632)
proxy_1                     | 	at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
proxy_1                     | 	at java.util.concurrent.ForkJoinPool$WorkQueue.runTask(ForkJoinPool.java:1056)
proxy_1                     | 	at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1692)
proxy_1                     | 	at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:157)
proxy_1                     | Caused by: com.mchange.v2.resourcepool.TimeoutException: A client timed out while waiting to acquire a resource from com.mchange.v2.resourcepool.BasicResourcePool@22de2f79 -- timeout at awaitAvailable()
proxy_1                     | 	at com.mchange.v2.resourcepool.BasicResourcePool.awaitAvailable(BasicResourcePool.java:1505)
proxy_1                     | 	at com.mchange.v2.resourcepool.BasicResourcePool.prelimCheckoutResource(BasicResourcePool.java:644)
proxy_1                     | 	at com.mchange.v2.resourcepool.BasicResourcePool.checkoutResource(BasicResourcePool.java:554)
proxy_1                     | 	at com.mchange.v2.c3p0.impl.C3P0PooledConnectionPool.checkoutAndMarkConnectionInUse(C3P0PooledConnectionPool.java:758)
proxy_1                     | 	at com.mchange.v2.c3p0.impl.C3P0PooledConnectionPool.checkoutPooledConnection(C3P0PooledConnectionPool.java:685)
proxy_1                     | 	... 10 more

MAINTAINER PSC "psc@georchestra.org"
LABEL maintainer="psc@georchestra.org"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch !

@RemiDesgrange
Copy link
Contributor Author

I cannot find any bug in master, I run tests (with IT) and launch the composition. No issue found.

@RemiDesgrange RemiDesgrange marked this pull request as ready for review September 28, 2020 10:13
@fvanderbiest fvanderbiest added this to the 20.1.0 milestone Sep 28, 2020
@fvanderbiest fvanderbiest merged commit 92e365d into master Sep 28, 2020
@fvanderbiest fvanderbiest deleted the 3018-upgrade-to-postgres-12 branch September 28, 2020 11:34
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

3 participants