Revert Finalize support for RightMark join + Mark join

[comphead]

…p (#16488)"

This reverts commit d73f0e8.

Which issue does this PR close?

• Closes Extended tests are failing #16595 .

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

[alamb]

FYI @jonathanc-n

[alamb]

Thank you @comphead -- I think it is a good idea to back this out and fix it for real when we are ready

[comphead]

Thank you @comphead -- I think it is a good idea to back this out and fix it for real when we are ready

we should probably ask to run extended tests if PR label is one of core, physical-expr, logical-expr, optimizer

[alamb]

Thank you @comphead -- I think it is a good idea to back this out and fix it for real when we are ready

we should probably ask to run extended tests if PR label is one of core, physical-expr, logical-expr, optimizer

Yeah, I think we never figured out how to actually trigger it via comments (there is some vestigal github action but i don't htink it works reliably)

[jonathanc-n]

@alamb @comphead How do you run extended tests?

[comphead]

@alamb @comphead How do you run extended tests?

I'm not sure if you can access it https://github.com/apache/datafusion/actions/workflows/extended.yml

[alamb]

I think you can just put a comment like this in your PR

Run extended tests

If not, you can also run it manually

https://github.com/apache/datafusion/tree/main/datafusion/sqllogictest#running-tests-sqlite