Skip to content

Commit 884512d

Browse files
committed
logs(worker): reduce logs
1 parent 183e712 commit 884512d

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
@@ -109,20 +109,6 @@ def update_status(self, new_status: str, time_spent: float = None, request_id: s
109109
self.status = new_status
110110
self.status_time = 0.0
111111

112-
logging.info(
113-
"Worker status update",
114-
extra={
115-
"worker": {
116-
"status": self.status,
117-
"request_id": self.processing_id,
118-
"requests_processed": self.requests_processed,
119-
"requests_failed": self.requests_failed,
120-
"total_idle_time": self.total_idle_time,
121-
"total_processing_time": self.total_processing_time,
122-
}
123-
},
124-
)
125-
126112
async def keep_alive(self) -> NoReturn:
127113
if self.queue is None:
128114
raise RuntimeError("queue was not initialised")
@@ -271,7 +257,7 @@ def process_request(
271257

272258
if datasource is None:
273259
# request.user_message += "Failed to process request."
274-
logging.info(request.user_message)
260+
logging.debug(request.user_message)
275261
raise Exception("Request was not accepted by any datasources.")
276262
else:
277263
request.user_message += "Success"
@@ -305,14 +291,14 @@ def on_request_complete(self) -> None:
305291
"""Called when the request processing exits cleanly"""
306292

307293
self.request.user_message = "Success" # Do not report log history on successful request
308-
logging.info("Request completed successfully.")
294+
logging.debug("Request completed successfully.")
309295
self.request_store.set_request_status(self.request, Status.PROCESSED)
310296
self.requests_processed += 1
311297

312298
def on_request_fail(self, exception: Exception) -> None:
313299
"""Called when the request processing raises an exception"""
314300

315-
logging.exception("Request failed with exception.")
301+
logging.debug("Request failed with exception.")
316302
error_message = self.request.user_message + "\n" + str(exception)
317303
self.request.user_message = error_message
318304
self.request_store.set_request_status(self.request, Status.FAILED)
@@ -330,5 +316,5 @@ def on_process_terminated(self) -> None:
330316
self.request_store.set_request_status(self.request, Status.QUEUED)
331317
self.queue.nack(self.queue_msg)
332318
else:
333-
logging.info("Worker shutting down", extra={"request": self.request.serialize()})
334319
self.request_store.update_request(self.request)
320+
logging.info("Worker shutting down")

0 commit comments

Comments
 (0)