Skip to content

Commit 87f9476

Browse files
committed
logs(worker): reduce logs
1 parent e30aad0 commit 87f9476

File tree

2 files changed

+5
-19
lines changed

2 files changed

+5
-19
lines changed

polytope_server/common/request.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ def set_status(self, value: Status) -> None:
9595
if self.status_history is None:
9696
self.status_history = {}
9797
self.status_history.setdefault(value.value, now_ts)
98-
logging.info("Request %s status set to %s.", self.id, value.value)
98+
logging.debug("Request %s status set to %s.", self.id, value.value)
9999

100100
@classmethod
101101
def serialize_slot(cls, key, value):

polytope_server/worker/worker.py

Lines changed: 4 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -107,20 +107,6 @@ def update_status(self, new_status: str, time_spent: float = None, request_id: s
107107
self.status = new_status
108108
self.status_time = 0.0
109109

110-
logging.info(
111-
"Worker status update",
112-
extra={
113-
"worker": {
114-
"status": self.status,
115-
"request_id": self.processing_id,
116-
"requests_processed": self.requests_processed,
117-
"requests_failed": self.requests_failed,
118-
"total_idle_time": self.total_idle_time,
119-
"total_processing_time": self.total_processing_time,
120-
}
121-
},
122-
)
123-
124110
async def keep_alive(self) -> NoReturn:
125111
if self.queue is None:
126112
raise RuntimeError("queue was not initialised")
@@ -265,7 +251,7 @@ def process_request(
265251

266252
if datasource is None:
267253
# request.user_message += "Failed to process request."
268-
logging.info(request.user_message)
254+
logging.debug(request.user_message)
269255
raise Exception("Request was not accepted by any datasources.")
270256
else:
271257
request.user_message += "Success"
@@ -299,14 +285,14 @@ def on_request_complete(self) -> None:
299285
"""Called when the request processing exits cleanly"""
300286

301287
self.request.user_message = "Success" # Do not report log history on successful request
302-
logging.info("Request completed successfully.")
288+
logging.debug("Request completed successfully.")
303289
self.request_store.set_request_status(self.request, Status.PROCESSED)
304290
self.requests_processed += 1
305291

306292
def on_request_fail(self, exception: Exception) -> None:
307293
"""Called when the request processing raises an exception"""
308294

309-
logging.exception("Request failed with exception.")
295+
logging.debug("Request failed with exception.")
310296
error_message = self.request.user_message + "\n" + str(exception)
311297
self.request.user_message = error_message
312298
self.request_store.set_request_status(self.request, Status.FAILED)
@@ -324,5 +310,5 @@ def on_process_terminated(self) -> None:
324310
self.request_store.set_request_status(self.request, Status.QUEUED)
325311
self.queue.nack(self.queue_msg)
326312
else:
327-
logging.info("Worker shutting down", extra={"request": self.request.serialize()})
328313
self.request_store.update_request(self.request)
314+
logging.info("Worker shutting down")

0 commit comments

Comments
 (0)