Homec4science

Audit - another partial fix to commit re-parsing bug

Authored by Bob Trahan <btrahan@phacility.com> on Oct 21 2014, 02:39.

Description

Audit - another partial fix to commit re-parsing bug

Summary: Ref T6350. We build $sql based on whose subscribed. If no one is subscribed, then we have no query to run.

Test Plan: observed one error disappearing from my daemon log. Also, more doing it live.

Reviewers: epriestley, chad

Reviewed By: chad

Subscribers: Korvin, epriestley

Maniphest Tasks: T6350

Differential Revision: https://secure.phabricator.com/D10731

Details

Committed
Bob Trahan <btrahan@phacility.com>Oct 21 2014, 02:39
Pushed
aubortJan 31 2017, 17:16
Parents
rPHec82ad2d16ed: Audit / Herald - make audits added via herald "required"
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <btrahan@phacility.com> committed rPHd248bc47cf00: Audit - another partial fix to commit re-parsing bug (authored by Bob Trahan <btrahan@phacility.com>).Oct 21 2014, 02:39