calcite icon indicating copy to clipboard operation
calcite copied to clipboard

[CALCITE-7301] Support unparse special syntax when operator is LAMBDA/MERGE/UNPIVOT

Open dssysolyatin opened this issue 1 month ago • 6 comments

dssysolyatin avatar Nov 19 '25 12:11 dssysolyatin

saw a useful comment from Julian on one of previous task:

It might be helpful for SqlUnparserTest to unparse everything before and after cloning. Could you enable that in this PR? Then you might not need a specific test case.

It would be good to add this to the current PR. Then we don't need such tests, I hope

dssysolyatin avatar Nov 19 '25 12:11 dssysolyatin

So, is your PR intended to accomplish what Julian mentioned?

xiedeyantu avatar Nov 19 '25 13:11 xiedeyantu

I added a test, but many operators in server package (ServerUnParserTest test) don’t have createCall. I’ll look into that next week.

Also, I’m not sure if I should add the operator at the end of getOperandList, or if I can reorder them to match the constructor parameter order. The last one would be much better. One example of this is UNPIVOT

dssysolyatin avatar Nov 19 '25 22:11 dssysolyatin

What is the status of this PR?

mihaibudiu avatar Dec 04 '25 01:12 mihaibudiu

I will most likely come back to this PR at the end of next week

dssysolyatin avatar Dec 06 '25 11:12 dssysolyatin