calcite icon indicating copy to clipboard operation
calcite copied to clipboard

[CALCITE-5787] Add interface in RelNode for getInputFieldsUsed

Open xiedeyantu opened this issue 1 month ago • 7 comments

See CALCITE-5787

xiedeyantu avatar Nov 26 '25 15:11 xiedeyantu

Sorry, it's still in draft form and not ready yet. Thank you very much for your comments—I'll carefully consider them.

xiedeyantu avatar Nov 26 '25 16:11 xiedeyantu

I noticed the email Julian sent; it seems we need to discuss this in an email.

Yes, I saw it. I replied in Jira. I think we should continue the discussion in Jira, email was just a temporary measure.

xiedeyantu avatar Nov 27 '25 02:11 xiedeyantu

I am not very sure how this will be used, but this looks fine. You are not covering all existing RelNode types yet.

mihaibudiu avatar Dec 01 '25 18:12 mihaibudiu

I am not very sure how this will be used, but this looks fine. You are not covering all existing RelNode types yet.

This JIRA requirement originates from CALCITE-5740, aiming to more accurately determine which columns from child nodes are actually utilized by parent nodes, thereby facilitating the judgment of whether a conversion to semi-join is possible.

xiedeyantu avatar Dec 01 '25 23:12 xiedeyantu

The requester of this PR has not provided a clear response, so it may be put on hold for an extended period.

xiedeyantu avatar Dec 06 '25 11:12 xiedeyantu

Hi @julianhyde, since you were the one who proposed this, could you please review this PR when you have time?

xiedeyantu avatar Dec 12 '25 01:12 xiedeyantu