Re: useless LIMIT_OPTION_DEFAULT

2023-12-16 Thread Alvaro Herrera
On 2023-Dec-14, Tom Lane wrote: > Zhang Mingli writes: > > By reading the codes, I found that we process limit option as > > LIMIT_OPTION_WITH_TIES when using WITH TIES > > and all others are LIMIT_OPTION_COUNT by  commit > > 357889eb17bb9c9336c4f324ceb1651da616fe57. > > And check actual limit

Re: useless LIMIT_OPTION_DEFAULT

2023-12-14 Thread Tom Lane
Zhang Mingli writes: > By reading the codes, I found that we process limit option as > LIMIT_OPTION_WITH_TIES when using WITH TIES > and all others are LIMIT_OPTION_COUNT by  commit > 357889eb17bb9c9336c4f324ceb1651da616fe57. > And check actual limit node in limit_needed(). > There is no need

useless LIMIT_OPTION_DEFAULT

2023-12-13 Thread Zhang Mingli
Hi, all By reading the codes, I found that we process limit option as LIMIT_OPTION_WITH_TIES when using WITH TIES and all others are LIMIT_OPTION_COUNT by  commit 357889eb17bb9c9336c4f324ceb1651da616fe57. And check actual limit node in limit_needed(). There is no need to maintain a useless