Re: Re: FETCH FIRST clause WITH TIES option

2019-03-31 Thread Surafel Temesgen
On Sun, Mar 31, 2019 at 3:14 AM Tomas Vondra wrote: > > > Hi, > > I got to look at the patch today, with the intent to commit, but sadly I > ran into a couple of minor issues that I don't feel comfortable fixing > on my own. Attached is a patch highlighling some of the places (0001 is > your v7 p

Re: Re: FETCH FIRST clause WITH TIES option

2019-03-30 Thread Tomas Vondra
On Sun, Mar 31, 2019 at 01:14:46AM +0100, Tomas Vondra wrote: On Fri, Mar 29, 2019 at 01:56:48AM +0100, Tomas Vondra wrote: On Tue, Mar 26, 2019 at 10:46:00AM +0300, Surafel Temesgen wrote: On Mon, Mar 25, 2019 at 11:56 AM David Steele wrote: This patch no longer passes testing so marked Wai

Re: Re: FETCH FIRST clause WITH TIES option

2019-03-30 Thread Tomas Vondra
On Fri, Mar 29, 2019 at 01:56:48AM +0100, Tomas Vondra wrote: On Tue, Mar 26, 2019 at 10:46:00AM +0300, Surafel Temesgen wrote: On Mon, Mar 25, 2019 at 11:56 AM David Steele wrote: This patch no longer passes testing so marked Waiting on Author. Thank you for informing. Fixed Thanks for

Re: Re: FETCH FIRST clause WITH TIES option

2019-03-28 Thread Tomas Vondra
On Tue, Mar 26, 2019 at 10:46:00AM +0300, Surafel Temesgen wrote: On Mon, Mar 25, 2019 at 11:56 AM David Steele wrote: This patch no longer passes testing so marked Waiting on Author. Thank you for informing. Fixed Thanks for the updated patch. I do have this on my list of patches that

Re: Re: FETCH FIRST clause WITH TIES option

2019-03-26 Thread Surafel Temesgen
On Mon, Mar 25, 2019 at 11:56 AM David Steele wrote: > This patch no longer passes testing so marked Waiting on Author. > > Thank you for informing. Fixed diff --git a/doc/src/sgml/ref/select.sgml b/doc/src/sgml/ref/select.sgml index 06d611b64c..b3b045ea87 100644 --- a/doc/src/sgml/ref/select.sg

Re: Re: FETCH FIRST clause WITH TIES option

2019-03-25 Thread David Steele
On 2/4/19 2:37 PM, Surafel Temesgen wrote: On Mon, Feb 4, 2019 at 8:29 AM Michael Paquier > wrote: This patch needs a rebase because of conflicts done recently for pluggable storage, so moved to next CF, waiting on author. -- Thank you . rebased agai