On Sep 20, 2010, at 6:48 AM, Kevan Miller wrote: > On Sep 17, 2010, at 5:31 PM, Jason Dillon wrote: >> I understand. I think simply logging the class names of the >> resources/providers as INFO should be sufficient. The other details raw >> type/data map (not even sure what that is) should probably be logged at >> DEBUG. > > Patches welcome... ;-)
Patches... we don't need to stinking patches. :-P I almost made one for you guys, but when I noticed the comments says that those logs were meant to be INFO I left it alone and simply configured logback to omit them from my logging. The logging may be useful when debugging, and a simple summary might be helpful to show folks where they have misconfigured their resources. Bug again I think of this as DEBUG logging, not something that one would really need/want in production code. Anyways, I could write a patch if needed... but I think whoever (i've not checked the commit logs) put the INFO + comments about INFO probably should look at these changes. --jason
