bugfix(pipeline): Fix state machine of hash join

[leiysky]

I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/

Summary

Fix the bug that hash join probe transform won't emit Finished event.

Changelog

• Bug Fix

Related Issues

Close #5234

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Updated
databend	⬜️ Ignored (Inspect)		May 9, 2022 at 4:42AM (UTC)

[mergify]

Thanks for the contribution!
I have applied any labels matching special text in your PR Changelog.

Please review the labels and make any necessary changes.

[BohuTANG]

LGTM.
Do we have any plan to move some logic to some files from sql/exec/mod.rs? The codes in it are getting more and more trivial :D

[leiysky]

LGTM. Do we have any plan to move some logic to some files from sql/exec/mod.rs? The codes in it are getting more and more trivial :D

It's just somewhat glue code. I'm going to do some decouple work on the concepts PlanNode, Expression and Transform after migrating to new planner. After that, we will deal with these files.

[BohuTANG]

Not sure why this is not working:

mysql> select * from numbers(1000) as t inner join numbers(1000) as t1 on t.number = t1.number;
ERROR 1105 (HY000): Code: 1002, displayText = Duplicate alias name :number (while in processor thread 0).

[BohuTANG]

Oh, this is OK:

mysql> select t.number from numbers(1000) as t inner join numbers(1000) as t1 on t.number = t1.number;

[sundy-li]

it's the same problem described in #5236