Subject: Re: Tid scan improvements
To: David Rowley <dgrowleyml@gmail.com>
Cc: Edmund Horner <ejrh00@gmail.com>,
 PostgreSQL Developers <pgsql-hackers@lists.postgresql.org>
References: 
 <CAMyN-kCX8kNy85_Sa-=gXNhZL84vbDtxpKPTF7qTDtTwykaOEQ@mail.gmail.com>
 <20191201023416.GV2355@paquier.xyz> <20210101013019.GS13234@fetter.org>
 <CAMyN-kCv4ekxAJDFM7H3yjNpcxr9OPSY3C=yrFODDSdL4vctww@mail.gmail.com>
 <CAApHDvqZcqVgWy1nY6xnQpVExtfx=DSKCdXbBxT_b5hQOgu2dA@mail.gmail.com>
 <CAApHDvozoYzeBqzVss_oh5HUFFSMZ2afmQQka-wH4xHU2LUE1w@mail.gmail.com>
 <20210203211919.u7q3vq24g4pd63gv@alap3.anarazel.de>
 <CAApHDvqDcx3_CM1Y4hiCv7q3o3iLFsF1g=SQ5B=6NJoSPEDA_Q@mail.gmail.com>
 <CAApHDvrdoop-9ojBMLX0x_xa0=6v_trhXM8soAwSb5yebaWPgg@mail.gmail.com>
 <CAApHDvor_H6qRrCHXpQKODZaS7L1hU5ogyBGGx3cxLiNgiDhdw@mail.gmail.com>
 <20210216220522.3sn64ui3prwm43zh@alap3.anarazel.de>
 <CAApHDvqfLgx4xaYcJg3wgnW-fe2B-qLUKt00rkPz7QMunQL6+A@mail.gmail.com>
 <CAApHDvqoO33Em0tWoJvwN2cqvAJ45nB+qqfS5cxLbQ45BGPwkg@mail.gmail.com>
 <CAApHDvrxaKMBrV-Tr3ogLaNJMSPC7_GGQWprsDg+HA3Ky0=m=g@mail.gmail.com>
From: Peter Eisentraut <peter.eisentraut@enterprisedb.com>
Message-ID: <05b070ee-b68c-56ca-be41-aa240ee62d81@enterprisedb.com>
Date: Mon, 7 Jun 2021 12:11:53 +0200
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:78.0)
 Gecko/20100101 Thunderbird/78.10.2
MIME-Version: 1.0
In-Reply-To: 
 <CAApHDvrxaKMBrV-Tr3ogLaNJMSPC7_GGQWprsDg+HA3Ky0=m=g@mail.gmail.com>
Content-Type: multipart/mixed;
 boundary="------------A9EDB56E93D75DDC22723D24"
Content-Language: en-US
Archived-At: 
 <https://www.postgresql.org/message-id/05b070ee-b68c-56ca-be41-aa240ee62d81%40enterprisedb.com>
Precedence: bulk

This is a multi-part message in MIME format.
--------------A9EDB56E93D75DDC22723D24
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit

This patch didn't add _outTidRangePath() even though we have outNode() 
coverage for most/all path nodes.  Was this just forgotten?  See 
attached patch.

--------------A9EDB56E93D75DDC22723D24
Content-Type: text/plain; charset=UTF-8; x-mac-type="0"; x-mac-creator="0";
 name="0001-Add-_outTidRangePath.patch"
Content-Transfer-Encoding: base64
Content-Disposition: attachment;
 filename="0001-Add-_outTidRangePath.patch"

RnJvbSAzYzY5NmY4MTJkNGM2ZjhjNjZiYzc1MTA1YzNjMWFmNzljM2IyOTIyIE1vbiBTZXAg
MTcgMDA6MDA6MDAgMjAwMQpGcm9tOiBQZXRlciBFaXNlbnRyYXV0IDxwZXRlckBlaXNlbnRy
YXV0Lm9yZz4KRGF0ZTogTW9uLCA3IEp1biAyMDIxIDEyOjA0OjQ5ICswMjAwClN1YmplY3Q6
IFtQQVRDSF0gQWRkIF9vdXRUaWRSYW5nZVBhdGgoKQoKV2UgaGF2ZSBvdXROb2RlKCkgY292
ZXJhZ2UgZm9yIGFsbCBwYXRoIG5vZGVzLCBidXQgdGhpcyBvbmUgd2FzCm1pc3NlZCB3aGVu
IGl0IHdhcyBhZGRlZC4KLS0tCiBzcmMvYmFja2VuZC9ub2Rlcy9vdXRmdW5jcy5jIHwgMTMg
KysrKysrKysrKysrKwogMSBmaWxlIGNoYW5nZWQsIDEzIGluc2VydGlvbnMoKykKCmRpZmYg
LS1naXQgYS9zcmMvYmFja2VuZC9ub2Rlcy9vdXRmdW5jcy5jIGIvc3JjL2JhY2tlbmQvbm9k
ZXMvb3V0ZnVuY3MuYwppbmRleCAwNDY5NmY2MTNjLi5lMzJiOTJlMjk5IDEwMDY0NAotLS0g
YS9zcmMvYmFja2VuZC9ub2Rlcy9vdXRmdW5jcy5jCisrKyBiL3NyYy9iYWNrZW5kL25vZGVz
L291dGZ1bmNzLmMKQEAgLTE4NTksNiArMTg1OSwxNiBAQCBfb3V0VGlkUGF0aChTdHJpbmdJ
bmZvIHN0ciwgY29uc3QgVGlkUGF0aCAqbm9kZSkKIAlXUklURV9OT0RFX0ZJRUxEKHRpZHF1
YWxzKTsKIH0KIAorc3RhdGljIHZvaWQKK19vdXRUaWRSYW5nZVBhdGgoU3RyaW5nSW5mbyBz
dHIsIGNvbnN0IFRpZFJhbmdlUGF0aCAqbm9kZSkKK3sKKwlXUklURV9OT0RFX1RZUEUoIlRJ
RFJBTkdFUEFUSCIpOworCisJX291dFBhdGhJbmZvKHN0ciwgKGNvbnN0IFBhdGggKikgbm9k
ZSk7CisKKwlXUklURV9OT0RFX0ZJRUxEKHRpZHJhbmdlcXVhbHMpOworfQorCiBzdGF0aWMg
dm9pZAogX291dFN1YnF1ZXJ5U2NhblBhdGgoU3RyaW5nSW5mbyBzdHIsIGNvbnN0IFN1YnF1
ZXJ5U2NhblBhdGggKm5vZGUpCiB7CkBAIC00MTY2LDYgKzQxNzYsOSBAQCBvdXROb2RlKFN0
cmluZ0luZm8gc3RyLCBjb25zdCB2b2lkICpvYmopCiAJCQljYXNlIFRfVGlkUGF0aDoKIAkJ
CQlfb3V0VGlkUGF0aChzdHIsIG9iaik7CiAJCQkJYnJlYWs7CisJCQljYXNlIFRfVGlkUmFu
Z2VQYXRoOgorCQkJCV9vdXRUaWRSYW5nZVBhdGgoc3RyLCBvYmopOworCQkJCWJyZWFrOwog
CQkJY2FzZSBUX1N1YnF1ZXJ5U2NhblBhdGg6CiAJCQkJX291dFN1YnF1ZXJ5U2NhblBhdGgo
c3RyLCBvYmopOwogCQkJCWJyZWFrOwotLSAKMi4zMS4xCgo=
--------------A9EDB56E93D75DDC22723D24--