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

	Unknown Object (File)
	Fri, Apr 5, 9:22 PM

	Unknown Object (File)
	Tue, Apr 2, 4:46 AM

	Unknown Object (File)
	Mar 31 2024, 11:25 AM

	Unknown Object (File)
	Mar 31 2024, 11:25 AM

	Unknown Object (File)
	Mar 20 2024, 4:51 PM

	Unknown Object (File)
	Jan 19 2024, 4:52 PM

	Unknown Object (File)
	Jan 15 2024, 5:21 PM

	Unknown Object (File)
	Jan 13 2024, 10:54 AM

View All 74 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