> How to you propose to handle the use of getChildExps in RelWriterImpl?

RelWriterImpl is using getChildExps _only_ to assert that all the
RexNodes were inserted in correct order (e.g. the order of .item calls
match getChildExps).

Is this a proper assertion at all?

I can emulate that behavior with RexShuttle and it won't hurt
performance much since it would execute only in assert-enabled build.
However I do not feel this assertion was properly placed in the first place.

Vladimir

Reply via email to