Apply suggestions from code review

Co-authored-by: Tian Xia <cblmemo@gmail.com>

sky/jobs/controller.py

@@ -439,7 +439,7 @@ def _cancel_all_tasks(self, task_id: int) -> None:
     def run(self) -> None:
         """Run controller logic and handle exceptions."""
         all_tasks_completed = lambda: self._num_tasks == len(self._task_status)
-        # TODO(andy):Serve has a logic to prevent from too many services running
+        # TODO(andy): Serve has a logic to prevent from too many services running
         # at the same time. We should have a similar logic here, but instead we
         # should calculate the sum of the subtasks (an upper bound), instead of
         # the number of jobs (dags).