Fix sending notifications in the successful case
[?]
Jun 23, 2015, 12:13 AM
7I7XHQAE62GIKLJ2JUBT4TYX4GTZRZVIQSDKRT52CM6SVUPNRDKACDependencies
- [2]
IE2PRAQUhydra-queue-runner: Send build notifications - [3]
HUUZFPPKFix race between the queue monitor and the builder threads - [4]
O7MWBXF6Doh - [5]
HPJKBFZ4Handle concurrent finishing of the same build - [6]
ENXUSMSVMake concurrency more robust - [7]
24BMQDZAStart of single-process hydra-queue-runner
Change contents
- edit in src/hydra-queue-runner/hydra-queue-runner.cc at line 1160
std::vector<BuildID> buildIDs; - edit in src/hydra-queue-runner/hydra-queue-runner.cc at line 1209
buildIDs.push_back(b->id); - replacement in src/hydra-queue-runner/hydra-queue-runner.cc at line 1213
/* Send notification about this build. */{auto notificationSenderQueue_(notificationSenderQueue.lock());notificationSenderQueue_->push(NotificationItem(build->id, std::vector<BuildID>()));/* Send notification about the builds that have this step asthe top-level. */for (auto id : buildIDs) {{auto notificationSenderQueue_(notificationSenderQueue.lock());notificationSenderQueue_->push(NotificationItem(id, std::vector<BuildID>()));}notificationSenderWakeup.notify_one(); - edit in src/hydra-queue-runner/hydra-queue-runner.cc at line 1222
notificationSenderWakeup.notify_one(); - edit in src/hydra-queue-runner/hydra-queue-runner.cc at line 1312
dependentIDs.push_back(build2->id); - edit in src/hydra-queue-runner/hydra-queue-runner.cc at line 1337
dependentIDs.push_back(b->id);