Re: [BUG v13] Crash with event trigger in extension
От | Jehan-Guillaume de Rorthais |
---|---|
Тема | Re: [BUG v13] Crash with event trigger in extension |
Дата | |
Msg-id | 20200904113728.2d4a44bb@firost обсуждение исходный текст |
Ответ на | [BUG v13] Crash with event trigger in extension (Jehan-Guillaume de Rorthais <jgdr@dalibo.com>) |
Ответы |
Re: [BUG v13] Crash with event trigger in extension
|
Список | pgsql-bugs |
On Fri, 4 Sep 2020 10:20:13 +0900 Michael Paquier <michael@paquier.xyz> wrote: > On Thu, Sep 03, 2020 at 07:41:14PM +0200, Jehan-Guillaume de Rorthais wrote: > > +$node->safe_psql('postgres', > > + q{CREATE EXTENSION test_event_trigger VERSION '1.0'}); > > +$node->safe_psql('postgres', > > + q{ALTER EXTENSION test_event_trigger UPDATE TO '2.0'}); > > + > > +my $ans = $node->safe_psql('postgres', q{ > > + SELECT extversion > > + FROM pg_catalog.pg_extension > > + WHERE extname='test_event_trigger' > > +}); > > + > > +is($ans, '2.0', 'Extension test_event_trigger upgraded'); > > I may be missing something of course, but if this just tests a > sequence of SQL commands, why don't you just include a SQL test in > text_extensions/sql/? That would be much cheaper than having to > create a full scenario with a TAP script. Indeed, it's much more simple and cheaper. However, the result is a bit confusing as it's not a wrong SQL result, but the backend that actually crash. The crash message itself appears in results/test_extensions.out, but no other tests are run. Neither from the same SQL file or from other ones. That Means the diff is quite large, but the error appears close from the top. I don't know if some other crash regression tests has been included that way in the past. But if this is OK, I can append the test to sql/test_extensions.sql and remove the TAP. Let me know what you think. Regards,
В списке pgsql-bugs по дате отправления: