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/03/02 03:50:46 UTC

[GitHub] [calcite] Aaaaaaron edited a comment on pull request #2253: [CALCITE-4385] Extracts and eliminates/merges the condition that has common expressions (Jiatao Tao)

Aaaaaaron edited a comment on pull request #2253:
URL: https://github.com/apache/calcite/pull/2253#issuecomment-788558782


   > Can we please set a hard limit on the maximum number of back-and-forth, and close/reject PRs that exceed it?
   > 
   > I see the PR does not meet code/performance quality, and I have no time to review it. I unsubscribe.
   
   @vlsi very sad to hear this, some of your review comment is great and I've worked hard to revise them, no one's code is perfect if you think it is still not meet code/performance, show the evidence, and please show more respect!


----------------------------------------------------------------
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.

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