Differential D19257

Drop uniqueness constraint on PushEvent request ID
ClosedPublic
Actions

Authored by epriestley on Mar 26 2018, 1:58 PM.

Tags

None

Referenced Files

	F18811413: D19257.id46103.diff
	Mon, Oct 20, 3:16 AM

	F18788862: D19257.id.diff
	Wed, Oct 15, 6:49 AM

	F18509472: D19257.id.diff
	Sep 5 2025, 3:25 AM

	F18502476: D19257.diff
	Sep 4 2025, 10:27 PM

	F18089933: D19257.id46103.diff
	Aug 6 2025, 1:24 PM

	F18089932: D19257.id46102.diff
	Aug 6 2025, 1:24 PM

	F18089930: D19257.id.diff
	Aug 6 2025, 1:24 PM

	F18086607: D19257.diff
	Aug 5 2025, 11:02 PM

View All Files

Subscribers

None

Details

Reviewers: None
Commits: rP5e710959bed9: (stable) Drop uniqueness constraint on PushEvent request ID
rPf583406ba93a: Drop uniqueness constraint on PushEvent request ID

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.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 26 2018, 1:58 PM

Harbormaster completed remote builds in B19921: Diff 46102.Mar 26 2018, 1:59 PM

epriestley requested review of this revision.Mar 26 2018, 1:59 PM

This revision was not accepted when it landed; it landed in state Needs Review.Mar 26 2018, 2:02 PM

Closed by commit rPf583406ba93a: Drop uniqueness constraint on PushEvent request ID (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

sql/

autopatches/

20180326.lock.03.nonunique.sql

2 lines

src/

applications/

repository/

storage/

PhabricatorRepositoryPushEvent.php

3 lines

Diff 46103

View Options

resources/sql/autopatches/20180326.lock.03.nonunique.sql