zhztheplayer opened a new pull request, #8033:
URL: https://github.com/apache/incubator-gluten/pull/8033

   Remove parameter "altConstraintSet` from RAS planner as it's not an optimal 
approach for root-level physical property enforcement. Perhaps a better way is 
to enrich "requiredChildrenConstraint" in property model API to support 
multiple constraint combinations, then passing a fake root with wanted 
constraint sets.
   
   And it's practical to remove the alternative constraint set immediately 
since it is not likely strictly required by Gluten, because Gluten converts 
Spark plan nodes to Gluten plan nodes through regular RAS rules than through 
enforcer rules.


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