Differential D21532 Diff 51252 src/applications/repository/worker/commitmessageparser/PhabricatorRepositoryCommitMessageParserWorker.php
Changeset View
Changeset View
Standalone View
Standalone View
src/applications/repository/worker/commitmessageparser/PhabricatorRepositoryCommitMessageParserWorker.php
Show All 18 Lines | if (!$this->shouldSkipImportStep()) { | ||||
$ref = $commit->newCommitRef($viewer); | $ref = $commit->newCommitRef($viewer); | ||||
$data = $this->loadCommitData($commit); | $data = $this->loadCommitData($commit); | ||||
$data->setCommitRef($ref); | $data->setCommitRef($ref); | ||||
$this->updateCommitData($commit, $data); | $this->updateCommitData($commit, $data); | ||||
} | } | ||||
if ($this->shouldQueueFollowupTasks()) { | $this->queueCommitTask($this->getFollowupTaskClass()); | ||||
$this->queueTask( | |||||
$this->getFollowupTaskClass(), | |||||
array( | |||||
'commitID' => $commit->getID(), | |||||
), | |||||
array( | |||||
// We queue followup tasks at default priority so that the queue | |||||
// finishes work it has started before starting more work. If | |||||
// followups are queued at the same priority level, we do all | |||||
// message parses first, then all change parses, etc. This makes | |||||
// progress uneven. See T11677 for discussion. | |||||
'priority' => PhabricatorWorker::PRIORITY_DEFAULT, | |||||
)); | |||||
} | |||||
} | } | ||||
final protected function updateCommitData( | final protected function updateCommitData( | ||||
PhabricatorRepositoryCommit $commit, | PhabricatorRepositoryCommit $commit, | ||||
PhabricatorRepositoryCommitData $data) { | PhabricatorRepositoryCommitData $data) { | ||||
$ref = $data->getCommitRef(); | $ref = $data->getCommitRef(); | ||||
$viewer = $this->getViewer(); | $viewer = $this->getViewer(); | ||||
▲ Show 20 Lines • Show All 83 Lines • Show Last 20 Lines |