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 2020/11/11 10:28:17 UTC

[GitHub] [calcite] rubenada commented on pull request #2254: [CALCITE-4390] Fix SqlMatchRecognize operand list

rubenada commented on pull request #2254:
URL: https://github.com/apache/calcite/pull/2254#issuecomment-725341151


   LGTM
   
   Thanks for the PR @dawidwys . Some minor details to correct the commit message (as per https://calcite.apache.org/develop/#contributing):
   - The commit message should describe the issue, and should not begin with "Fix ...". IMHO the Jira title ("SqlMatchRecognize returns wrong operand list") would be a more adequate commit message.
   - The commit message should end with your name & last name in parentheses.


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