Re: Modify an incorrect regression test case in the group by key value elimination function

On Tue, 18 Feb 2025 at 20:40, songjinzhou <tsinghualucky912(at)foxmail(dot)com> wrote:
> Two months ago, we enhanced the group by key value elimination function. This is a very useful function. When I was learning, I found a regression test case that was not suitable, as follows:

> Because the table structure of t3 is defined as follows(Its PK is deferrable):

> I think this test case does not fully reflect the original meaning. So I made a small change to this and look forward to your feedback. Thanks!

That's my mistake. Thanks for spotting it.

I looked at your patch and felt that the flow would be better if all
those tests used t2 rather than t3. That way we don't need to add the
NOT NULL constraint to both tables, just t2. I wrote the patch to do
it that way and pushed it.

David