mrhhsg opened a new pull request, #19758:
URL: https://github.com/apache/doris/pull/19758

   # Proposed changes
   
   Refactoring the filtering conditions in the current ExecNode from an 
expression tree to an array can simplify the process of adding runtime filters. 
It eliminates the need for complex merge operations and removes the requirement 
for the frontend to combine expressions into a single entity.
   
   By representing the filtering conditions as an array, each condition can be 
treated individually, making it easier to add runtime filters without the need 
for complex merging logic. The array can store the individual conditions, and 
the runtime filter logic can iterate through the array to apply the filters as 
needed.
   
   This refactoring simplifies the codebase, improves readability, and reduces 
the complexity associated with handling filtering conditions and adding runtime 
filters. It separates the conditions into discrete entities, enabling more 
straightforward manipulation and management within the execution node.
   
   ## Problem summary
   
   Describe your changes.
   
   ## Checklist(Required)
   
   * [ ] Does it affect the original behavior
   * [ ] Has unit tests been added
   * [ ] Has document been added or modified
   * [ ] Does it need to update dependencies
   * [ ] Is this PR support rollback (If NO, please explain WHY)
   
   ## Further comments
   
   If this is a relatively large or complex change, kick off the discussion at 
[[email protected]](mailto:[email protected]) by explaining why you 
chose the solution you did and what alternatives you considered, etc...
   
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to