On Tue, Sep 29, 2009 at 10:21 AM, Jay Pipes <[email protected]> wrote:

>
> I actually think that the better (as far as encapsulation goes) is to call
> logging_pre_do() after Session::readAndStoreQuery().
>Then again, I think
> logging_post_do() should be moved into a Session-level context instead of
> being in the parser where it currently is...

Did you mean logging_pre_do() should be moved into a session context ?
(or maybe both?)

In order to be able to rewrite the query text, I think I'd have to
either move logging_pre_do() inside Session::readAndStoreQuery(),
right before the

return true line, or call logging_pre_do()  inside sql_parser.cc
inside   case COM_QUERY: (but then it is not at the sesson-level :( )

Any takes?

  -Diego

>
> -jay
>



-- 
Diego Medina
Web Developer
http://www.fmpwizard.com

_______________________________________________
Mailing list: https://launchpad.net/~drizzle-discuss
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~drizzle-discuss
More help   : https://help.launchpad.net/ListHelp

Reply via email to