search

kasei / attean

A Perl Semantic Web Framework
19 stars 10 forks source link

Trace logging in prune_plans #40

Closed kjetilk closed 8 years ago

kjetilk commented 8 years ago

Just a start of what we discussed in #19. I added the trace logging to prune_plans, to help debug my invalid plans problem.

Would also be interesting to hear if this is at all a good idea.

Kjetil

kasei commented 8 years ago

Want to add MooX::Log::Any to the prereqs?

kjetilk commented 8 years ago

Sure!

kasei commented 8 years ago

Should we look more deeply into the reasons jonas avoided this module in favor of MooX::Role::Logger? (This uncertainty is the reason I haven't dealt with logging in Attean yet!)

kjetilk commented 8 years ago

We might... But the MooX::Log::Any source code is very, very straightforward. Not much that can go wrong in there. I imagine that the consuming application may want more from their logging, so the question is 1) whether we would want more, and I don't think so, 2) whether it makes sense to use the more complex module anyway for some concern that the consuming application might have.

Actually, I'm inclined to not spend more time on it and just go for the simplest module.