On Mon, Mar 31, 2014 at 02:06:28PM -0400, Bruce Momjian wrote:
> Where are we on this? It seem odd that psql sends /* */ comments to the
> server, but not "--" comments. Should this be documented or changed?
>
> I am confused why changing the behavior would affect the regression test
> output as
On Sat, Dec 28, 2013 at 08:20:59PM -0500, Tom Lane wrote:
> Andres Freund writes:
> > On 2013-12-24 12:27:59 -0500, Tom Lane wrote:
> >> What I was proposing was that we do include comments in what we send,
> >> as long as those comments are embedded in the statement text, not
> >> on lines before
Andres Freund writes:
> On 2013-12-24 12:27:59 -0500, Tom Lane wrote:
>> What I was proposing was that we do include comments in what we send,
>> as long as those comments are embedded in the statement text, not
>> on lines before it.
> The common way I've seen what I've described above done as i
On 2013-12-24 12:27:59 -0500, Tom Lane wrote:
> Andres Freund writes:
> >> This is inconsistent, IMO. I think if we were to fix things so that
> >> leading block comments were dropped the same way -- comments are, that
> >> would also take care of the behavior complained of in this thread.
> >> T
On 12/24/2013 08:53 AM, Martijn van Oosterhout wrote:
(Untested). Isn't this just a case of psql not printing out a timing if
the server responds with PGRES_EMPTY_QUERY?
Yes, it is. Sorry should have made myself more clear (way more clear
when I read my messages from yesterday). Then I thought
Andres Freund writes:
>> This is inconsistent, IMO. I think if we were to fix things so that
>> leading block comments were dropped the same way -- comments are, that
>> would also take care of the behavior complained of in this thread.
>> There's been some previous discussion of this point, I th
> The real question is whether we shouldn't suppress the whole PQexec.
> I believe this is very closely related to the question of what we do
> with a comment preceding the next command. Try this experiment:
> regression=# /* block comment here */
> regression-# select 2+2;
> regression=# -- das
Andres Freund writes:
> Maybe I am thinking to technical here, but why would that be a good
> idea? After all, the comment will have triggered sending a statement to
> the server and waiting for the result. The user might want to know about
> that.
I agree; if we triggered a server operation, we
On Tue, December 24, 2013 15:19, Andres Freund wrote:
> Hi,
>
> On 2013-12-24 02:05:23 +0100, Erik Rijkers wrote:
>> With \timing on, a trailing comment yields a timing.
>>
>> # test.sql
>> select 1;
>>
>> /*
>> select 2
>> */
>>
>> $ psql -f test.sql
>> ?column?
>> --
>> 1
>> (1 r
Hi,
On 2013-12-24 02:05:23 +0100, Erik Rijkers wrote:
> With \timing on, a trailing comment yields a timing.
>
> # test.sql
> select 1;
>
> /*
> select 2
> */
>
> $ psql -f test.sql
> ?column?
> --
> 1
> (1 row)
>
> Time: 0.651 ms
> Time: 0.089 ms
>
> I assume it is timing so
On 12/24/13, 5:40 AM, FabrÃzio de Royes Mello wrote:
>> (Untested). Isn't this just a case of psql not printing out a timing if
>> the server responds with PGRES_EMPTY_QUERY?
>>
>
> Works... look to the attached patch!
That looks reasonable.
--
Sent via pgsql-hackers mailing list (pgsql-hacker
On Tue, Dec 24, 2013 at 5:53 AM, Martijn van Oosterhout
wrote:
>
> On Tue, Dec 24, 2013 at 03:40:58AM +0100, Andreas Karlsson wrote:
> > On 12/24/2013 03:17 AM, David Johnston wrote:
> > It is not sent to the server as a trailing comment. The following
> > file is sent to the server like this.
> >
On Tue, Dec 24, 2013 at 03:40:58AM +0100, Andreas Karlsson wrote:
> On 12/24/2013 03:17 AM, David Johnston wrote:
> It is not sent to the server as a trailing comment. The following
> file is sent to the server like this.
>
> File:
> /**/;
> /**/
>
> Commands:
> PQexec(..., "/**/;");
> PQexec(...
On 12/24/2013 03:17 AM, David Johnston wrote:
I need to be convinced that the server should not just silently ignore
trailing comments. I'd consider an exception if the only text sent is a
comment ( in such a case we should throw an error ) but if valid commands
are sent and there is just some c
Andreas Karlsson wrote
> On 12/24/2013 02:05 AM, Erik Rijkers wrote:
>> With \timing on, a trailing comment yields a timing.
>>
>> # test.sql
>> select 1;
>>
>> /*
>> select 2
>> */
>>
>> $ psql -f test.sql
>> ?column?
>> --
>> 1
>> (1 row)
>>
>> Time: 0.651 ms
>> Time: 0.089 ms
On 12/24/2013 02:05 AM, Erik Rijkers wrote:
With \timing on, a trailing comment yields a timing.
# test.sql
select 1;
/*
select 2
*/
$ psql -f test.sql
?column?
--
1
(1 row)
Time: 0.651 ms
Time: 0.089 ms
I assume it is timing something about that comment (right?).
Confusi
16 matches
Mail list logo