GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/2875
[CARBONDATA-3038] Refactor dynamic configuration
1.remove CarbonProperty annotation, and DynamicConfigurable,
NonDynamicConfigurable, DefaultValue,Threshold and so on
2.unify default value variable name: XX_XX_DEFAULT
3.optimize the properties: remove some warning
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:
- [ ] Any interfaces changed?
No
- [ ] Any backward compatibility impacted?
No
- [ ] Document update required?
change the doc
- [ ] Testing done
no
- [ ] For large changes, please consider breaking it into sub-tasks under
an umbrella JIRA.
no
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/xubo245/carbondata
CARBONDATA-3038_RefactorConfiguration
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/carbondata/pull/2875.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2875
----
commit 436746f923f90d9db23c6a53a30e39d97a88042d
Author: xubo245 <xubo29@...>
Date: 2018-10-29T11:09:15Z
[CARBONDATA-3038] Refactor dynamic configuration
1.remove CarbonProperty annotation, and DynamicConfigurable,
NonDynamicConfigurable, DefaultValue,Threshold and so on
2.unify default value variable name: XX_XX_DEFAULT
3.optimize the properties: remove some warning
optmize
----
---
