Excessive logging belongs to debug mode [#1227640]

Node Gallery logs too many non-important events to watchdog. For example, cache clear for a gallery is not important enough to log it every time. Because default logging method of Drupal is database, and DB write is expensive operation, we should avoid excessive logging.
IMO, events like these should be only logged when debugging / troubleshooting.

Comment	File	Size	Author
#2	Loging settings.PNG	72.47 KB	dddave

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #0.0

crea CreditAttribution: crea commented 23 July 2011 at 17:01

Issue summary:

View changes

better wording

Comment #1

dddave CreditAttribution: dddave commented 24 July 2011 at 13:27

I considered opening an issue for this myself when I noticed all the "cache" entries in my watchdog.

Comment #2

dddave CreditAttribution: dddave commented 26 July 2011 at 09:03

File	Size
Loging settings.PNG	72.47 KB

Attaching a pic of a similar option for font-your-face module in D7.

Comment #3

justintime CreditAttribution: justintime commented 27 July 2011 at 16:42

Nah, they just need removed altogether. They were added there when I was developing the caching layer, and I haven't needed to look for those messages in months. If I were to keep them there (and I'm not), I'd convert them to drush_log() statements -- who uses the UI to clear caches anymore? :)

Comment #4

scroogie CreditAttribution: scroogie commented 27 July 2011 at 17:03

I do. I always have the devel block enabled, and its just quicker most of the time to use the handy link. :P

Nevertheless agree with the change though. Apparently I always ignored the calls. Shows you how good it is to have some additional eyes on the code sometimes.