Skip to content

Update DB recommendations#5793

Merged
Alkarex merged 2 commits intoFreshRSS:edgefrom
Alkarex:doc-postgresql
Oct 31, 2023
Merged

Update DB recommendations#5793
Alkarex merged 2 commits intoFreshRSS:edgefrom
Alkarex:doc-postgresql

Conversation

@Alkarex
Copy link
Copy Markdown
Member

@Alkarex Alkarex commented Oct 31, 2023

Favour PostgreSQL, based on tests such as:
#5648 (comment)
#5707

SQLite is generally very fine, although with the challenge that the DB model cannot easily be updated https://www.sqlite.org/lang_altertable.html#why_alter_table_is_such_a_problem_for_sqlite
(We should ship a command-line update mechanism)

I have verified that FreshRSS still works with mysql:5.5.62
https://hub.docker.com/layers/library/mysql/5.5.62/images/sha256-d404d78aa797c87c255e5ae2beb5d8d0e4d095f930b1f20dc208eaa957477b74?context=explore

@Alkarex Alkarex added this to the 1.23.0 milestone Oct 31, 2023
Copy link
Copy Markdown
Member

@Frenzie Frenzie left a comment

Choose a reason for hiding this comment

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

What's that about SQLite being built into PHP?

@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Oct 31, 2023

What's that about SQLite being built into PHP?

I am not sure how to formulate that better. It does not require installing anything else like a DB server, as the support comes from PHP (with the appropriate extensions) itself.

@Frenzie
Copy link
Copy Markdown
Member

Frenzie commented Oct 31, 2023

Maybe something more in the vein of easy setup?

@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Oct 31, 2023

It is not even easy setup, it is no setup at all :-)

@Frenzie
Copy link
Copy Markdown
Member

Frenzie commented Oct 31, 2023

Eh, you have to apt/apk install it and you have to set it up in FreshRSS. :-)

@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Oct 31, 2023

Eh, you have to apt/apk install it and you have to set it up in FreshRSS. :-)

Right, indeed for the package. I was more thinking from the Docker perspective.

I was curious of how much that package was adding (on debian:12-slim after Apache and PHP are installed):

apt install php-sqlite3
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
The following additional packages will be installed:
  php8.2-sqlite3
The following NEW packages will be installed:
  php-sqlite3 php8.2-sqlite3
0 upgraded, 2 newly installed, 0 to remove and 0 not upgraded.
Need to get 33.8 kB of archives.
After this operation, 145 kB of additional disk space will be used.

Regarding the setup in FreshRSS, there is nothing to do (besides picking SQLite, of course) :-)

image

@Frenzie
Copy link
Copy Markdown
Member

Frenzie commented Oct 31, 2023

I was more thinking from the Docker perspective.

Then maybe "included in Docker image"?

Regarding the setup in FreshRSS, there is nothing to do (besides picking SQLite, of course) :-)

That's what I meant, yes. :-D

@Alkarex
Copy link
Copy Markdown
Member Author

Alkarex commented Oct 31, 2023

I have changed a bit the explanations. SQLite being included in our Docker images is also mentioned in the Docker documentation, so that is probably enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants