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/09/15 02:19:33 UTC

[GitHub] [calcite] xy2953396112 opened a new pull request #2520: [CALCITE-4177] Throw exception when deserialize SqlOperator fails, do…

xy2953396112 opened a new pull request #2520:
URL: https://github.com/apache/calcite/pull/2520


   … not return null. (Xzh & Wang Yanlin)


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



[GitHub] [calcite] xy2953396112 commented on pull request #2520: [CALCITE-4177] Throw exception when deserialize SqlOperator, do…

Posted by GitBox <gi...@apache.org>.
xy2953396112 commented on pull request #2520:
URL: https://github.com/apache/calcite/pull/2520#issuecomment-920971793


   > @xy2953396112 Please modify the PR commit info about "Throw exception when deserialize SqlOperator fails" maybe " "Throw exception when deserialize SqlOperator" is better?
   
   thanks for review, update the commit message.


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



[GitHub] [calcite] hsyuan closed pull request #2520: [CALCITE-4177] Throw exception when deserialize SqlOperator, do…

Posted by GitBox <gi...@apache.org>.
hsyuan closed pull request #2520:
URL: https://github.com/apache/calcite/pull/2520


   


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



[GitHub] [calcite] xy2953396112 edited a comment on pull request #2520: [CALCITE-4177] Throw exception when deserialize SqlOperator fails, do…

Posted by GitBox <gi...@apache.org>.
xy2953396112 edited a comment on pull request #2520:
URL: https://github.com/apache/calcite/pull/2520#issuecomment-920716991


   @Julianhyde @yanlin-Lynn @zabetak @hsyuan 
   Please help review this pr when you have time.


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



[GitHub] [calcite] NobiGo commented on pull request #2520: [CALCITE-4177] Throw exception when deserialize SqlOperator fails, do…

Posted by GitBox <gi...@apache.org>.
NobiGo commented on pull request #2520:
URL: https://github.com/apache/calcite/pull/2520#issuecomment-920851475


   @xy2953396112  Please modify the PR commit info about "Throw exception when deserialize SqlOperator fails"  maybe   " "Throw exception when deserialize SqlOperator" is better?


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



[GitHub] [calcite] xy2953396112 commented on pull request #2520: [CALCITE-4177] Throw exception when deserialize SqlOperator fails, do…

Posted by GitBox <gi...@apache.org>.
xy2953396112 commented on pull request #2520:
URL: https://github.com/apache/calcite/pull/2520#issuecomment-920716991


   @Julian Hyde @yanlin-Lynn @Stamatis Zampetakis
   Please help review this pr when you have time.


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



[GitHub] [calcite] xy2953396112 edited a comment on pull request #2520: [CALCITE-4177] Throw exception when deserialize SqlOperator fails, do…

Posted by GitBox <gi...@apache.org>.
xy2953396112 edited a comment on pull request #2520:
URL: https://github.com/apache/calcite/pull/2520#issuecomment-920716991


   @Julianhyde @yanlin-Lynn @zabetak 
   Please help review this pr when you have time.


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