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 2019/12/26 12:46:57 UTC

[GitHub] [calcite] danny0405 opened a new pull request #1697: [CALCITE-3635] Supports hint option as string or numeric literal

danny0405 opened a new pull request #1697: [CALCITE-3635] Supports hint option as string or numeric literal
URL: https://github.com/apache/calcite/pull/1697
 
 
   

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


With regards,
Apache Git Services

[GitHub] [calcite] danny0405 edited a comment on issue #1697: [CALCITE-3635] Supports hint option as string or numeric literal

Posted by GitBox <gi...@apache.org>.
danny0405 edited a comment on issue #1697: [CALCITE-3635] Supports hint option as string or numeric literal
URL: https://github.com/apache/calcite/pull/1697#issuecomment-569159521
 
 
   > I am thinking if we can directly extend the option as a general literal than limiting to string or numeric literal?
   
   I didn't found any hint that needs literal option of other format, like TIMESTAMP, i'm cautious to add new syntax because i don't want to "create requests".

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


With regards,
Apache Git Services

[GitHub] [calcite] danny0405 merged pull request #1697: [CALCITE-3635] Supports hint option as string or numeric literal

Posted by GitBox <gi...@apache.org>.
danny0405 merged pull request #1697: [CALCITE-3635] Supports hint option as string or numeric literal
URL: https://github.com/apache/calcite/pull/1697
 
 
   

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


With regards,
Apache Git Services

[GitHub] [calcite] amaliujia edited a comment on issue #1697: [CALCITE-3635] Supports hint option as string or numeric literal

Posted by GitBox <gi...@apache.org>.
amaliujia edited a comment on issue #1697: [CALCITE-3635] Supports hint option as string or numeric literal
URL: https://github.com/apache/calcite/pull/1697#issuecomment-569136355
 
 
   I am thinking if we can directly extend the option as a general literal than limiting to string or numeric literal?
   
   

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


With regards,
Apache Git Services

[GitHub] [calcite] danny0405 commented on issue #1697: [CALCITE-3635] Supports hint option as string or numeric literal

Posted by GitBox <gi...@apache.org>.
danny0405 commented on issue #1697: [CALCITE-3635] Supports hint option as string or numeric literal
URL: https://github.com/apache/calcite/pull/1697#issuecomment-569159521
 
 
   > I am thinking if we can directly extend the option as a general literal than limiting to string or numeric literal?
   
   I didn't found any hint that needs literal optional of other format, like TIMESTAMP, i'm cautious to add new syntax because i don't want to "create requests".

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


With regards,
Apache Git Services

[GitHub] [calcite] amaliujia commented on issue #1697: [CALCITE-3635] Supports hint option as string or numeric literal

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #1697: [CALCITE-3635] Supports hint option as string or numeric literal
URL: https://github.com/apache/calcite/pull/1697#issuecomment-569136355
 
 
   I am thinking if we can directly extend the option as a general literal than limit to string or numeric literal?

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


With regards,
Apache Git Services