goto / compass

Metadata storage service
https://goto.github.io/compass/
Apache License 2.0
2 stars 0 forks source link

perf: index asset_urn, timestamp cols in asset_probes tbl #55

Closed sudo-suhas closed 10 months ago

sudo-suhas commented 10 months ago

By default, golang-migrate wraps multiple SQL statements in a transaction. Dropping index concurrently is not allowed in a transaction. So the drop statement needs to be the only statement in the migration.

These indexes are necessary because we order by and filter the rows in asset_probes with these columns:

coveralls commented 10 months ago

Pull Request Test Coverage Report for Build 5922869460


Totals Coverage Status
Change from base Build 5875638947: 0.0%
Covered Lines: 6194
Relevant Lines: 7101

💛 - Coveralls