Send less event updates for judgements in parallel mode.

We used to send event updates for all testcases that came in after we already determined the final verdict for a problem. Now we check if we already have a previous end time for the judgement, and then we don't do this anymore.

Note that race conditions can still make it such that we send multiple update events, where two judgehosts know the result at the same time and. However, this happens way less often.

Author: nickygerritsen

webapp/src/Controller/API/JudgehostController.php

@@ -1023,9 +1023,11 @@ private function addSingleJudgingRun(
             if (!$hasNullResults || $lazyEval !== DOMJudgeService::EVAL_FULL) {
                 // NOTE: setting endtime here determines in testcases_GET
                 // whether a next testcase will be handed out.
+                $this->logger->error('Judging %d is done, setting endtime, it had %d', [ $judging->getJudgingid(), $judging->getEndtime() ]);
+                $this->logger->error('Verdict is %s', [$result]);
+                $sendJudgingEvent = !$judging->getEndtime();
                 $judging->setEndtime(Utils::now());
                 $this->maybeUpdateActiveJudging($judging);
-                $sendJudgingEvent = true;
             }
             $this->em->flush();