search

bgruening / docker-galaxy

:whale::bar_chart::books: Docker Images tracking the stable Galaxy releases.
http://bgruening.github.io/docker-galaxy
MIT License
226 stars 134 forks source link

moving to Postgresql 11 #504

Closed gmauro closed 5 years ago

gmauro commented 5 years ago

Migrating both versions, Monolithic and Compose, to Postgresql 11.2

bgruening commented 5 years ago

@gmauro thanks a lot. But it seems the mono one is failing its test.

pcm32 commented 5 years ago

What is the rationale for moving to Postgres 11 so soon? Are we constrained by functionality in 10 that is present in 11? I think we only recently moved to 10. What is the current preferred postgres version to run Galaxy with? Thanks!

bgruening commented 5 years ago

What is the rationale for moving to Postgres 11 so soon?

PG 11 was released 6 months ago I think. Is that too early?

Are we constrained by functionality in 10 that is present in 11?

I don't understand this question? But I don't think Galaxy is constrained by any functionality in 10 or 11.

I think we only recently moved to 10

Who is "we"? The Galaxy Docker project moved to 10 only in the dev branch and we had an open issue/question if we should straight move to 11.

What is the current preferred postgres version to run Galaxy with?

I don't think we recommend any version. However, one feature that is only available in "newer" version I think (10 and 11) is the table locks used by handler assignments. But this is optional and we can use this with 10 as well.

Our thinking is that we would like to avoid database upgrades for our users. If we do the switch from 9 - 11 we should be save for the next 3-4 years. Next LTS Ubuntu will most likely also have PG 11. So might also change the base image to 20.04 as soon as it's released without worrying too much about database upgrades.

Is that a good reason? Any reason to stick to PG 10 from your side?

bgruening commented 5 years ago

Cool thanks @gmauro! This looks great!

bgruening commented 5 years ago

Not sure what travis problem is currently. I will merge and work further on the dev branch. Thanks @gmauro!