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

Telemetry: Database prefixes #6747

Open
yorkshire-pudding opened this issue Nov 8, 2024 · 2 comments
Open

Telemetry: Database prefixes #6747

yorkshire-pudding opened this issue Nov 8, 2024 · 2 comments

Comments

@yorkshire-pudding
Copy link
Member

yorkshire-pudding commented Nov 8, 2024

Description of the need

Database prefixes have been part of Drupal for (?) years and were carried over into Backdrop. Most hosting that I've seen has no limit on numbers of databases, but the size of databases can sometimes be a problem, whether by limits from hosts or when exporting/importing. Exporting/importing individual sites from a database using prefixes requires more code.

If we added telemetry to measure the use of database prefixes, it could inform whether this "feature" could be removed/deprecated in Backdrop 2.0 any future features/development involving this area.

EDIT: @hosef said in Zulip:

Joseph Flatt: I don't think removing them is a good idea. Any change that removes functionality available in D7 is a huge mistake and makes it so that people who have more complicated sites will not consider using Backdrop.

Joseph Flatt: Also, Simpletest is totally dependent on database prefixes, so we would have to rebuild our testing infrastructure if we did that.

Proposed solution

We should track if sites are using database prefixes (Yes/No) as a Telemetry data-point.

@yorkshire-pudding yorkshire-pudding changed the title Add telemetry to find out how many, if any, Backdrop sites are using database prefixes Telemetry: Database prefixes Nov 8, 2024
@bugfolder
Copy link

I'm all for telemetry that informs future changes, but I will note that it's not uncommon for CiviCRM users to put the CiviCRM db into its own db, and then if you want to use modules that query both B and Civi dbs (e.g., CiviCRM Webforms), you need to (or at least, can) use prefixes. So removing/deprecating prefixes would break some sites that use a separate Civi db, including backdropcms.org.

@quicksketch
Copy link
Member

It never hurts to have more data on our site configurations. I'd be in favor of collecting this simple boolean as well. You never know when you'll get a surprising number.

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

No branches or pull requests

3 participants