[ 
https://issues.apache.org/jira/browse/CALCITE-3458?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16963359#comment-16963359
 ] 

Julian Hyde commented on CALCITE-3458:
--------------------------------------

We should encourage people to use {{toString()}} rather than 
{{getDescription()}}. Then they can pass the {{RelNode}} to the logger, and the 
logger can lazily call {{toString()}}. In fact, we should deprecate 
{{getDescription()}}.

> Remove desc in AbstractRelNode
> ------------------------------
>
>                 Key: CALCITE-3458
>                 URL: https://issues.apache.org/jira/browse/CALCITE-3458
>             Project: Calcite
>          Issue Type: Improvement
>          Components: core
>            Reporter: Haisheng Yuan
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> If the query is super large, e.g. contains tens of thousands of nodes or 
> expressions, the RelNode digest and desc become very large. The content of 
> desc and digest are almost the same, except that desc consists of id plus 
> digest, which causes OOM in our production system. So remove desc, just use 
> {{id + digest}} to produce description.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to