You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@calcite.apache.org by GitBox <gi...@apache.org> on 2021/07/21 04:48:01 UTC

[GitHub] [calcite] devozerov commented on pull request #2438: [CALCITE-4650] TopDownRuleDriver.DeriveTrait.derive produces wrong AssertionError when registering new nodes (Vladimir Ozerov)

devozerov commented on pull request #2438:
URL: https://github.com/apache/calcite/pull/2438#issuecomment-883884868


   Hi @vlsi @hsyuan @chunweilei,
   It's been a while since I opened this PR. I do not have a reproducer anymore. The problem appeared in the private codebase in one of the TPC-DS tests with a certain set of rules and a concrete `derive/passThrough` logic. Since then there were multiple changes and I no longer can reproduce the error. I cannot prove that it was a real problem with a planner as opposed to a problem with my own logic. Therefore, closing the PR and the ticket for now. Will re-open if needed.


-- 
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: commits-unsubscribe@calcite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org