search

babenkoivan / elastic-migrations

Elasticsearch migrations for Laravel
MIT License
187 stars 32 forks source link

Add primary key to elastic migrations table #48

Closed beblife closed 1 year ago

beblife commented 1 year ago

Currently the table that keeps track of the already migrated Elastic migrations has no primary key defined in the schema. This can be required in MySQL through sql_require_primary_key and is not always configurable by the end user (e.g. DigitalOcean enforces this on their managed database solution).

This PR adds a primary key to that table so the table can be created.

babenkoivan commented 1 year ago

Hey @beblife, thank you for pointing this out! I've added a primary key in v3.2.0. I think the migration column is more appropriate as a primary key.