-
Notifications
You must be signed in to change notification settings - Fork 466
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CORE][umbrella] Configuration enhancements #8326
Comments
@yikf as discussed in the offline, there are 2 requirement
|
@yikf Is the enhancement finished? Is there a readme about how to add a configuration? |
It's not completed yet. I will keep following up in the next few days. Eventually, there will be a configuration.md file to describe the configuration. Regarding the description of how to add the configuration, I can add it in this file. |
Description
As discussed in #6970, we plan to enhance the gluten configuration, and the specific enhancement solutions can be found in section #6970 (comment).
This issue serves as an umbrella to record the task list that we need to include to achieve our goals, because enhance config involves a large number of files, splitting multiple tasks facilitates code review and reduces the risk of introducing other problems.
GlutenConfig.getConf
toGlutenConfig.get
#8395GlutenConfig.scala
toorg.apache.gluten.config
package dir #8426ConfigEntry
to gluten config #8431The text was updated successfully, but these errors were encountered: