Tom Lane wrote:
> Bruce Momjian <> writes:
> > One thing you did was to log debug_query_string, but I don't see how
> > that could be the right value.  I assume it would be empty in a
> > client-side execute, or be the previous query.  I instead used "EXECUTE
> > portal_name" because that is what we are passed from the client.
> You could use portal->sourceText whenever that isn't null; it should be
> the querystring that the portal was created by.

Interesting, but right now we don't display the query using EXECUTE:
        test=> SET client_min_messages='log';
        test=> SET log_statement = 'all';
        test=> PREPARE xx AS SELECT 1;
        LOG:  statement: PREPARE xx AS SELECT 1;
        test=> EXECUTE xx;
        LOG:  statement: EXECUTE xx;
        (1 row)

so I assume client-side EXECUTE shouldn't either.  Do people want it
displayed?  The patch actually hard-codes the word EXECUTE after
"statement:" and prints the portal name:

        LOG:  statement: EXECUTE xx

  Bruce Momjian                        |               |  (610) 359-1001
  +  If your life is a hard drive,     |  13 Roberts Road
  +  Christ can be your backup.        |  Newtown Square, Pennsylvania 19073

---------------------------(end of broadcast)---------------------------
TIP 7: don't forget to increase your free space map settings

Reply via email to