#1862 closed defect (fixed)
Unneeded debug logs
| Reported by: | erikos | Owned by: | alsroot |
|---|---|---|---|
| Priority: | Unspecified by Maintainer | Milestone: | |
| Component: | journal | Version: | Unspecified |
| Severity: | Minor | Keywords: | r+ |
| Cc: | Distribution/OS: | Unspecified | |
| Bug Status: | Unconfirmed |
Description
src/jarabe/journal/model.py:188: logging.debug('cache hit and no need to grow the cache')
src/jarabe/journal/model.py:111: logging.debug('ResultSet.read position: %r', self._position)
I think we can remove those debug logs. They clutter the logs and don't bring as much value, imho.
Attachments (1)
Change History (8)
comment:1 Changed 13 years ago by tomeu
- Component changed from sugar to journal
- Owner changed from tomeu to alsroot
comment:2 follow-up: ↓ 3 Changed 13 years ago by erikos
- Keywords r! added; r? removed
Aleksey can you comment if you agree on my assumption that those logs are not necessary? If yes I will do a patch of course.
comment:3 in reply to: ↑ 2 Changed 13 years ago by alsroot
Replying to erikos:
Aleksey can you comment if you agree on my assumption that those logs are not necessary? If yes I will do a patch of course.
Agree, these messages are pretty annoying. If there is need to have it in released sugar, we can add special debug flag.
comment:4 Changed 13 years ago by erikos
- Keywords r? added; r! removed
comment:5 Changed 13 years ago by alsroot
- Keywords r+ added; r? removed
comment:6 Changed 13 years ago by erikos
- Resolution set to fixed
- Status changed from new to closed
Note: See
TracTickets for help on using
tickets.
missing patch?