ionelmc / python-redis-lock

Lock context manager implemented via redis SET NX EX and BLPOP.
https://pypi.python.org/pypi/python-redis-lock
BSD 2-Clause "Simplified" License
551 stars 77 forks source link

Lower the level of "{script} not cached." log #61

Closed suligap closed 5 years ago

suligap commented 6 years ago

It's nothing abnormal nor actionable. It can be just an info log instead.

ionelmc commented 6 years ago

Uh ... are warnings a problem for your deployment? I would like to understand why.

suligap commented 6 years ago

Sometimes people set up some sort of alerts for messages logged with WARNING or higher levels. I am interested in being (asynchronously) notified about WARNINGs. I'm not interested in warnings that are not actionable, for which I can't perform any action to fix them. This message right here is something that simply will happen and I as the operator or developer can't do anything about it. I might be interested in this event, but INFO or even DEBUG is more fitting in this case IMHO.