Skip to content

Add id to searchmeter_recent table#13

Open
takotakot wants to merge 1 commit into
bennettmcelwee:masterfrom
takotakot:add_id
Open

Add id to searchmeter_recent table#13
takotakot wants to merge 1 commit into
bennettmcelwee:masterfrom
takotakot:add_id

Conversation

@takotakot
Copy link
Copy Markdown
Contributor

No description provided.

@bennettmcelwee
Copy link
Copy Markdown
Owner

Thanks for your contributions. What is the purpose of adding the ID to this table?

@takotakot
Copy link
Copy Markdown
Contributor Author

I'm using MariaDB Galera Cluster for DB clustering. One of the limitations is about primary key:

All tables should have a primary key (multi-column primary keys are supported). DELETE operations are unsupported on tables without a primary key. Also, rows in tables without a primary key may appear in a different order on different nodes.

as described in Known Limitations.

If we add primary key (ID), throughput may decrease a little. But there is a (possibly old) problem that a table without a primary key may cause delay in replication [Delayed row-based replication with large tables lacking a primary key][without primary key 1][without primary key 2], too.

That's why I added the ID to this table.
Thanks.

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.

2 participants