You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by GitBox <gi...@apache.org> on 2021/09/06 10:33:22 UTC

[GitHub] [incubator-doris] ccoffline opened a new issue #6575: [Proposal] Refactor ProcNodeInterface to return ShowResultSet and remove ProcResult

ccoffline opened a new issue #6575:
URL: https://github.com/apache/incubator-doris/issues/6575


   ### Search before asking
   
   - [X] I had searched in the [issues](https://github.com/apache/incubator-doris/issues?q=is%3Aissue) and found no similar issues.
   
   
   ### Description
   
   Due to the interface design mistake, proc nodes all need to be evaluated twice, once to get the meta and once for the real calculation.
   `ProcResult` is redundant with `ShowResultSet`. So this issue want to merge into one and remove redundant calculations.
   
   ### Use case
   
   _No response_
   
   ### Related issues
   
   #6477
   
   ### Are you willing to submit PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
   


-- 
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@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org