leon

Results 8 comments of leon

> @Leon-WTF You might want to wait until #2602 is merged before you continue this. @cgivre Thanks for the info, #2602 is very needed, I will focus on implementing the...

> @Leon-WTF Is this ready for review? @cgivre Not yet, I'm handling the EXCEPT case, it needs to remove the duplicate records for probe side(for other three cases, it's like...

> @Leon-WTF I don't know if you saw this or not, but #2602 has been merged. We are actually running on Calcite 1.32 now! @cgivre Sorry for the late response....

> HI @Leon-WTF I saw that you added unit tests. Is this PR ready for review? If not, what is remaining? @cgivre I think it's just still need more ut...

@paul-rogers Thanks paul for your knowledge, but for the example of the serialized sort and merge, I don't quite understand. Why the sort and merge are in the same major...

@paul-rogers Thanks paul for your detailed explanation. One more question, why scan2 send to other sorts also blocked when scan2 send to sort1 is blocked? Do you think it's better...

@vvysotskyi Hi, any more comments on this PR?

> Hey @Leon-WTF Any chance you could address @vvysotskyi 's comments soon. This is one of the last PRs slated to be merged for the next release. I will do...