Re: Named Operators

2023-01-20 Thread Gurjeet Singh
On Sat, Jan 14, 2023 at 6:14 AM Gurjeet Singh wrote: > > I agree that an identifier _surrounded_ by the same token (e.g. #foo#) > or the pairing token (e.g. {foo}) looks better aesthetically, so I am > okay with any of the following variations of the scheme, as well: > > \#foo\# (tested; works)

Re: [PATCH] Teach planner to further optimize sort in distinct

2023-01-20 Thread Ankit Kumar Pandey
On 20/01/23 06:07, David Rowley wrote: Looking at the patch, you've not added any additional tests. If the existing tests are all passing then that just tells me that either the code is not functioning as intended or we have no tests that look at the EXPLAIN output which can make use of

Re: Named Operators

2023-01-20 Thread Ted Yu
On Fri, Jan 20, 2023 at 9:17 AM Gurjeet Singh wrote: > On Sat, Jan 14, 2023 at 6:14 AM Gurjeet Singh wrote: > > > > I agree that an identifier _surrounded_ by the same token (e.g. #foo#) > > or the pairing token (e.g. {foo}) looks better aesthetically, so I am > > okay with any of the following

<    1   2