Skip to content

Commit

Permalink
Merge pull request #99 from lsst/tickets/DM-45725
Browse files Browse the repository at this point in the history
DM-45725: Change default table engine to MyISAM
  • Loading branch information
JeremyMcCormick authored Aug 16, 2024
2 parents a4a3dd8 + 266d708 commit d24166b
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 3 additions & 0 deletions docs/user-guide/databases.rst
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,9 @@ To create a MySQL database from a schema file, the command would look similar to
In this case, the database would already need to have been created or the command will fail.

Tables in MySQL will by default use the ``MyISAM`` storage engine, which does not support foreign keys.
The engine can be changed in the table object of the schema by setting ``mysql:engine`` to ``InnoDB`` or another valid table engine name.

PostgreSQL
^^^^^^^^^^

Expand Down
2 changes: 1 addition & 1 deletion python/felis/datamodel.py
Original file line number Diff line number Diff line change
Expand Up @@ -492,7 +492,7 @@ class Table(BaseObject):
tap_table_index: int | None = Field(None, alias="tap:table_index")
"""IVOA TAP_SCHEMA table index of the table."""

mysql_engine: str | None = Field(None, alias="mysql:engine")
mysql_engine: str | None = Field("MyISAM", alias="mysql:engine")
"""MySQL engine to use for the table."""

mysql_charset: str | None = Field(None, alias="mysql:charset")
Expand Down

0 comments on commit d24166b

Please sign in to comment.