|From:||Andrey Borodin <x4mmm(at)yandex-team(dot)ru>|
|To:||Anastasia Lubennikova <a(dot)lubennikova(at)postgrespro(dot)ru>|
|Subject:||Re: [HACKERS] WIP: Covering + unique indexes.|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
> 16 янв. 2018 г., в 21:50, Anastasia Lubennikova <a(dot)lubennikova(at)postgrespro(dot)ru> написал(а):
> Updated patches are attached.
I've looked into the code, but haven't found anything broken.
Since I've tried to rebase patch myself and failed on parse utils, I've spend some cycles trying to break parsing.
One minor complain (no need to fix).
This is fine
x4mmm=# create index on pgbench_accounts (bid) include (aid,filler,upper(filler));
ERROR: expressions are not supported in included columns
But why not same error here? Previous message is very descriptive.
x4mmm=# create index on pgbench_accounts (bid) include (aid,filler,aid+1);
ERROR: syntax error at or near "+"
This works. But should not, IMHO
x4mmm=# create index on pgbench_accounts (bid) include (aid,aid,aid);
Do not know what's that...
# create index on pgbench_accounts (bid) include (aid desc, aid asc);
All these things allow foot-shooting with a small caliber, but do not break big things.
Unfortunately, amcheck_next does not work currently on HEAD (there are problems with AllocSetContextCreate() signature), but I've tested bt_index_check() before, during and after pgbench, on primary and on slave. Also, I've checked bt_index_parent_check() on master.
During bt_index_check() test from time to time I was observing
ERROR: canceling statement due to conflict with recovery
DETAIL: User query might have needed to see row versions that must be removed.
[install]check[-world] passed :)
From my POV, patch is in a good shape.
I think it is time to make the patch ready for committer again.
Best regards, Andrey Borodin.
|Next Message||Etsuro Fujita||2018-01-17 09:02:49||Re: [HACKERS] postgres_fdw bug in 9.6|
|Previous Message||Noah Misch||2018-01-17 08:36:31||Re: [HACKERS] Deadlock in XLogInsert at AIX|