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/02/24 08:03:20 UTC

[GitHub] [calcite] amaliujia opened a new pull request #1824: [CALCITE-3599] Add type to RexRangeRef's toString for debugging

amaliujia opened a new pull request #1824: [CALCITE-3599] Add type to RexRangeRef's toString for debugging
URL: https://github.com/apache/calcite/pull/1824
 
 
   @chunweilei Will it better to include type into debug string?

----------------------------------------------------------------
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 #1824: [CALCITE-3599] Add type to RexRangeRef's toString for debugging

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #1824: [CALCITE-3599] Add type to RexRangeRef's toString for debugging
URL: https://github.com/apache/calcite/pull/1824#issuecomment-590467707
 
 
   It might worth having `toString(verbose)` then to have a way to control if debug string is simple or full version. Can leave it for the future.

----------------------------------------------------------------
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 closed pull request #1824: [CALCITE-3599] Add type to RexRangeRef's toString for debugging

Posted by GitBox <gi...@apache.org>.
amaliujia closed pull request #1824: [CALCITE-3599] Add type to RexRangeRef's toString for debugging
URL: https://github.com/apache/calcite/pull/1824
 
 
   

----------------------------------------------------------------
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] chunweilei commented on issue #1824: [CALCITE-3599] Add type to RexRangeRef's toString for debugging

Posted by GitBox <gi...@apache.org>.
chunweilei commented on issue #1824: [CALCITE-3599] Add type to RexRangeRef's toString for debugging
URL: https://github.com/apache/calcite/pull/1824#issuecomment-590209455
 
 
   > @chunweilei Will it better to include type into debug string?
   
   Better not. Because the type string is really long.

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