On Thu, 10 Jun 2021 at 03:05, Thomas <thomasmannhart97@gmail.com> wrote: > We have implemented the Range Merge Join algorithm by extending the > existing Merge Join to also support range conditions, i.e., BETWEEN-AND > or @> (containment for range types).
It shouldn't be a blocker for you, but just so you're aware, there was a previous proposal for this in [1] and a patch in [2]. I've include Jeff here just so he's aware of this. Jeff may wish to state his intentions with his own patch. It's been a few years now.
I only just glanced over the patch. I'd suggest getting rid of the /* Thomas */ comments. We use git, so if you need an audit trail about changes then you'll find it in git blame. If you have those for an internal audit trail then you should consider using git. No committer would commit those to PostgreSQL, so they might as well disappear.
For further review, please add the patch to the July commitfest [3]. We should be branching for pg15 sometime before the start of July. There will be more focus on new patches around that time. Further details in [4].
Also, I see this if your first post to this list, so welcome, and thank you for the contribution. Also, just to set expectations; patches like this almost always take a while to get into shape for PostgreSQL. Please expect a lot of requests to change things. That's fairly standard procedure. The process often drags on for months and in some less common cases, years.
