Skip to content

Commit

Permalink
check for worker registration
Browse files Browse the repository at this point in the history
  • Loading branch information
1ntEgr8 committed Nov 27, 2024
1 parent 98a5247 commit 618348a
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions rpc/service.py
Original file line number Diff line number Diff line change
Expand Up @@ -321,8 +321,8 @@ async def DeregisterDriver(self, request, context):
async def RegisterTaskGraph(self, request, context):
stime = self.__stime()

if not self.__framework_registered():
msg = f"[{stime}] Trying to register a task graph (id={request.id}, name={request.name}) but no framework has been registered yet."
if not self.__worker_registered():
msg = f"[{stime}] Failed to register task graph (id={request.id}, name={request.name}) because no worker has been registered yet."
self._logger.error(msg)
return erdos_scheduler_pb2.RegisterTaskGraphResponse(
success=False, message=msg, num_executors=0
Expand Down Expand Up @@ -672,6 +672,11 @@ def __stime(self) -> EventTime:
def __framework_registered(self):
return self._simulator is not None

def __worker_registered(self):
return (
self.__framework_registered() and len(self.__get_worker_pool().workers) > 0
)

def __get_worker_pool(self):
# Simulator maintains only one worker pool, so this should be fine
return next(iter(self._simulator._worker_pools.worker_pools))
Expand Down

0 comments on commit 618348a

Please sign in to comment.