Task #5129: drop MariaDB support - Pulp

Actions

Send by e-mail Copy link

Task #5129

closed

drop MariaDB support

Added by dkliban@redhat.com over 5 years ago. Updated about 5 years ago.

Status:

CLOSED - CURRENTRELEASE

Priority:

Normal

Assignee:

daviddavis

Category:

Sprint/Milestone:

3.0.0

Start date:

Due date:

% Done:

100%

Estimated time:

Platform Release:

Groomed:

Yes

Sprint Candidate:

Tags:

Sprint:

Sprint 59

Quarter:

Description

The reasoning for dropping MariaDB support was summarized on pulp-dev list[0].

[0] https://www.redhat.com/archives/pulp-dev/2019-July/msg00035.html

Related issues

Added by dkliban@redhat.com over 5 years ago

Revision 75abd0fb | View on GitHub

Problem: testing against mariadb is not needed

Solution: remove mariadb option from plugin-template

re: #5129 https://pulp.plan.io/issues/5129

Actions

Copy link

Updated by daviddavis over 5 years ago

Groomed changed from No to Yes
Sprint set to Sprint 56

Actions

Copy link

Updated by amacdona@redhat.com over 5 years ago

Blocks Task #5130: Remove maria/mysql from installer added

Actions

Copy link

Updated by dalley over 5 years ago

In Pulpcore and Plugins, we should change CharField back to TextField where applicable. The only reason we used CharField was because MySQL cannot have an index on an unbounded text field.

In PostgreSQL this is not a limitation. There's no reason to use a 'varchar' type instead of a 'text' type unless you want a limit on the length of the text. They have identical performance (except for the overhead of the length check on 'varchar')

We should decide whether having a length limit on the field provides any value and remove it if not.

I don't see a compelling reason to switch back to autoincrement PKs despite the fact that we only moved away from them to begin with due to MySQL support.

Actions

Copy link

Updated by daviddavis over 5 years ago

It was also discussed that we should create a blog post for why we're dropping support. I think this is a good idea so I'm adding it to the checklist.