Skip to content

Commit 50ecdf8

Browse files
committed
feat: add backlog functionality logging
1 parent 261a7d8 commit 50ecdf8

File tree

2 files changed

+20
-0
lines changed

2 files changed

+20
-0
lines changed

packets/packet_queue.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,10 @@
11
# Copyright (C) Code Partners Pty. Ltd. All rights reserved. #
2+
import logging
3+
24
from packets.packet import Packet
35

6+
logger = logging.getLogger(__name__)
7+
48

59
class PacketQueueItem:
610
def __init__(self, packet: Packet):
@@ -32,6 +36,7 @@ def push(self, packet: Packet) -> None:
3236

3337
self.__count += 1
3438
self.__size += packet.size + self.__OVERHEAD
39+
logger.debug("Added packet {} to PacketQueue. Queue size is: {}".format(id(packet), self.__size))
3540
self.__resize()
3641

3742
def pop(self) -> (Packet, None):
@@ -50,6 +55,7 @@ def pop(self) -> (Packet, None):
5055

5156
self.__count -= 1
5257
self.__size -= packet.size + self.__OVERHEAD
58+
logging.debug("Popped packet %d" % id(packet))
5359
return packet
5460

5561
def clear(self) -> None:
@@ -58,6 +64,7 @@ def clear(self) -> None:
5864

5965
def __resize(self) -> None:
6066
while self.__backlog < self.__size:
67+
logger.debug("Queue size exceeds limit set by backlog.queue option (%d). Popping packet." % self.__backlog)
6168
if self.pop() is None:
6269
self.__size = 0
6370
break

protocols/protocol.py

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -266,8 +266,18 @@ def _impl_write_packet(self, packet: Packet) -> None:
266266
if self.__backlog_enabled:
267267
if (level.value >= self.__backlog_flushon.value and
268268
level != Level.CONTROL):
269+
270+
logger.debug("Packet level {} >= backlog flushon level {}. Flushing queue.".format(
271+
level,
272+
self.__backlog_flushon))
273+
269274
self.__flush_queue()
270275
else:
276+
logger.debug("Packet level {} < backlog flushon level {}. Pushing packet {} to queue.".format(
277+
level,
278+
self.__backlog_flushon,
279+
id(packet)))
280+
271281
self.__queue.push(packet)
272282
skip = True
273283

@@ -385,10 +395,13 @@ def __flush_queue(self) -> None:
385395
def __forward_packet(self, packet: Packet, disconnect: bool) -> None:
386396
if not self._connected:
387397
if not self.__keep_open:
398+
logger.debug("Protocol is not connected. Keep open is {}. Connecting.".format(self.__keep_open))
399+
388400
self._internal_connect()
389401
self._connected = True
390402
self.__failed = False
391403
else:
404+
logger.debug("Protocol is not connected. Keep open is {}. Reconnecting.".format(self.__keep_open))
392405
self.__do_reconnect()
393406

394407
if self._connected:

0 commit comments

Comments
 (0)