revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-17 20:32:41 +01:00
Code Issues Releases Wiki Activity

Drop uniqueness constraint on PushEvent request ID

Browse source 
Summary: See <https://discourse.phabricator-community.org/t/pushing-to-mercurial-repository-fails/1275/1>. Mercurial may invoke hooks multiple times per push.

Test Plan: Pushed to Mercurial, saw key constraint failure.

Differential Revision: https://secure.phabricator.com/D19257
This commit is contained in:
epriestley 2018-03-26 06:55:17 -07:00
parent bba1b185f8
commit f583406ba9
2 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
resources/sql/autopatches/20180326.lock.03.nonunique.sql Normal file
View file

@ -0,0 +1,2 @@
ALTER TABLE {$NAMESPACE}_repository.repository_pushevent
DROP KEY `key_request`;

3
src/applications/repository/storage/PhabricatorRepositoryPushEvent.php
View file

@ -46,9 +46,8 @@ final class PhabricatorRepositoryPushEvent
'key_repository' => array( 'key_repository' => array(
'columns' => array('repositoryPHID'), 'columns' => array('repositoryPHID'),
), ),
'key_request' => array( 'key_identifier' => array(
'columns' => array('requestIdentifier'), 'columns' => array('requestIdentifier'),
'unique' => true,
), ),
), ),
) + parent::getConfiguration(); ) + parent::getConfiguration();