For evaluation of bind joins the implementation for quite some time
makes use of a VALUES clause query.
Except for one code-path: for bind joins - where in the join all
arguments are bound - it was still using the old UNION query approach.
This approach is error prone and no longer required, i.e. the check join
can be executed with the same logic as the regular VALUES clause.
Note: an additional unit test for covering bind joins with no free vars
is added.
This change also marks a number of methods and classes used for the old
UNION based approach as deprecated. The implementations are internal to
the FedX engine and can be removed in the next major release.
GitHub issue resolved: #GH-5151
For evaluation of bind joins the implementation for quite some time makes use of a VALUES clause query.
Except for one code-path: for bind joins - where in the join all arguments are bound - it was still using the old UNION query approach. This approach is error prone and no longer required, i.e. the check join can be executed with the same logic as the regular VALUES clause.
Note: an additional unit test for covering bind joins with no free vars is added.
This change also marks a number of methods and classes used for the old UNION based approach as deprecated. The implementations are internal to the FedX engine and can be removed in the next major release.
PR Author Checklist (see the contributor guidelines for more details):
mvn process-resourcesto format from the command line)