yikf commented on PR #8920: URL: https://github.com/apache/incubator-gluten/pull/8920#issuecomment-2747400698
> Thanks for your pr! So there will be 6 configuration files for the two backends: GlutenConfig, GlutenStaticConfig, VeloxConfig, VeloxStaticConfig, CHConfig & CHStaticConfig, which makes me feel a bit complex. Could you explain the benefit of doing such separation? Static configuration typically implies that runtime modifications are not allowed, and it is made clearer through separation. We roughly discussed this point during the GlutenConfig optimization [at here](https://github.com/apache/incubator-gluten/issues/6970#issuecomment-2306067931). -- 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]
