silundong commented on PR #4619: URL: https://github.com/apache/calcite/pull/4619#issuecomment-3506141507
Perhaps wrapping the D (`dedupFreeVarsNode`) to make it immutable after creation would be better. Did I understand you correctly? The current draft implements the general decorrelation approach. For some very simple cases, the paper also proposes a simpler decorrelation that would yield cleaner plans. It seems that matching some fixed patterns using rules would suffice. I think the simple decorrelation approach can be considered later. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
