On Sun, Sep 8, 2019 at 11:35 PM Alexander Korotkov <a.korot...@postgrespro.ru> wrote: > I'm going to push 0001 changing "attno >= 1" to assert.
Pushed. Rebased patchset is attached. I propose to limit consideration during this commitfest to this set of 7 remaining patches. The rest of patches could be considered later. I made some minor editing in preparation to commit. But I decided I've couple more notes to Nikita. * 0003 extracts part of fields from BTScanOpaqueData struct into new BTScanStateData struct. However, there is a big comment regarding BTScanOpaqueData just before definition of BTScanPosItem. It needs to be revised. * 0004 adds "knnState" field to BTScanOpaqueData in addition to "state" field. Wherein "knnState" might unused during knn scan if it could be done in one direction. This seems counter-intuitive. Could we rework this to have "forwardState" and "backwardState" fields instead of "state" and "knnState"? ------ Alexander Korotkov Postgres Professional: http://www.postgrespro.com The Russian Postgres Company
0002-Enable-ORDER-BY-operator-scans-on-ordered-indexe-v15.patch.gz
Description: GNU Zip compressed data
0001-Introduce-ammorderbyopfirstcol-v15.patch.gz
Description: GNU Zip compressed data
0003-Extract-structure-BTScanState-v15.patch.gz
Description: GNU Zip compressed data
0004-Add-kNN-support-to-btree-v15.patch.gz
Description: GNU Zip compressed data
0005-Add-btree-distance-operators-v15.patch.gz
Description: GNU Zip compressed data
0006-Remove-distance-operators-from-btree_gist-v15.patch.gz
Description: GNU Zip compressed data
0007-Add-regression-tests-for-kNN-btree-v15.patch.gz
Description: GNU Zip compressed data