From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | Re: pgsql: Fix ts_headline() edge cases for empty query and empty search te |
Date: | 2023-04-06 21:38:13 |
Message-ID: | 20230406213813.uep7plg6lvcywujo@awork3.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Hi,
On 2023-04-06 19:52:55 +0000, Tom Lane wrote:
> Fix ts_headline() edge cases for empty query and empty search text.
Unfortunately there appears to be some portability issue with this. Causes a
test output difference on macos. Noticed it when testing something I was about
to push, and I was very confused for a second how my change could have a
portability aspect :)
https://cirrus-ci.com/build/6629557554380800
diff -U3 /Users/admin/pgsql/src/test/regress/expected/tsearch.out /Users/admin/pgsql/build/testrun/regress/regress/results/tsearch.out
--- /Users/admin/pgsql/src/test/regress/expected/tsearch.out 2023-04-06 20:03:11
+++ /Users/admin/pgsql/build/testrun/regress/regress/results/tsearch.out 2023-04-06 20:05:56
@@ -2133,6 +2133,9 @@
NOTICE: text-search query doesn't contain lexemes: ""
LINE 2: '', ''::tsquery);
^
+NOTICE: text-search query doesn't contain lexemes: ""
+LINE 2: '', ''::tsquery);
+ ^
ts_headline
-------------
Sure looks like somehow there's duplicate NOTICEs?
Greetings,
Andres Freund
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2023-04-06 21:47:32 | Re: pgsql: Fix ts_headline() edge cases for empty query and empty search te |
Previous Message | Tom Lane | 2023-04-06 21:33:44 | pgsql: psql: set SHELL_ERROR and SHELL_EXIT_CODE in more places. |