1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-12-22 13:30:55 +01:00
phorge-phorge/scripts
epriestley ee1e2da8fb Avoid Timeline race condition
Summary: While I should fix the transactional stuff, that patch is going to be
tricky and transactions have some performance implications. This is a simple
fix which prevents the race.

Instead of having the data point at the event ID, have the event point at a
data ID. Insert the data first, then insert the event with the right data
pointer. This is super simple and prevents the race issue.

Test Plan:
  - Ran the schema upgrade script, verified that the database was
    correctly upgraded. Was also prompted to stop daemons.
  - Ran 'repository-launch-master', verified that the discovery daemons were
    able to discover new commits and insert events for them. Verified the
    committask daemon was consuming events and converting them into tasks.
  - Verified new tasks looked correct in the database.
  - Browsed web interface.

Reviewers: jungejason

CC: tuomaspelkonen

Differential Revision: 133
2011-04-14 10:12:10 -07:00
..
daemon Move 'phd parse-commit' to a dedicated script; allow message parsing tasks to 2011-04-02 13:23:22 -07:00
repository Commit + Herald integration. 2011-04-04 14:13:14 -07:00
search Rought cut of search. 2011-02-14 15:34:20 -08:00
sql Avoid Timeline race condition 2011-04-14 10:12:10 -07:00
user Basic create_user script plus rough system agent support. 2011-04-12 18:34:53 -07:00
__init_env__.php Commit + Herald integration. 2011-04-04 14:13:14 -07:00
__init_script__.php Commit + Herald integration. 2011-04-04 14:13:14 -07:00
celerity_mapper.php Some herald improvements. 2011-03-22 20:43:19 -07:00