<p>Interestingly, due to my fat-fingering, it already picked up that the most recent migration (adding indexes to acls) could have been improved by creating them concurrently, instead of locking the acls table for the duration of the index build. Probably not a big deal for that table, but it would have been an easy fix, and would come in more useful for the larger tables too.</p>
<p><a href="https://github.com/openstreetmap/openstreetmap-website/blob/master/db/migrate/20190518115041_add_acl_indexes.rb">https://github.com/openstreetmap/openstreetmap-website/blob/master/db/migrate/20190518115041_add_acl_indexes.rb</a></p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/2237?email_source=notifications&email_token=AAK2OLMNBTLWS56P2O7KAYTPXZVWLA5CNFSM4HQLG52KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWPC5II#issuecomment-496905889">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAK2OLPBMI6FVTOIOORLSWLPXZVWLANCNFSM4HQLG52A">mute the thread</a>.<img src="https://github.com/notifications/beacon/AAK2OLLYS7ZHOWTPD7RQIFTPXZVWLA5CNFSM4HQLG52KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWPC5II.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/openstreetmap/openstreetmap-website/pull/2237?email_source=notifications\u0026email_token=AAK2OLMNBTLWS56P2O7KAYTPXZVWLA5CNFSM4HQLG52KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWPC5II#issuecomment-496905889",
"url": "https://github.com/openstreetmap/openstreetmap-website/pull/2237?email_source=notifications\u0026email_token=AAK2OLMNBTLWS56P2O7KAYTPXZVWLA5CNFSM4HQLG52KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWPC5II#issuecomment-496905889",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>