GitHub user akashrn5 opened a pull request:
https://github.com/apache/carbondata/pull/2375
[CARBONDATA-2585][CARBONDATA-2586][Local Dictionary]Support adding local
dictionary configuration in create table statement and show the configs in
describe formatted table
## What changes were proposed in this pull request?
In this PR, in order to support local dictionary, create table changes are
made to support local dictionary configurations as table properties and showing
those properties in describe formatted command based on whether the local
dictionary enabled or disabled.
Highlights:
basically we will have four properties
1. LOCAL_DICT_ENABLE => whether to enable or disable local dictionary
2. LOCAL_DICT_THRESHOLD => threshold property for the column to generate
local dictionary
3. LOCAL_DICT_INCLUDE => columns for which local dictionary needs to be
generated
4. LOCAL_DICT_EXCLUDE => columns for which local dictionary should not be
generated
## How was this patch tested?
Manual testing, and UTs are added in another PR.
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:
- [x] Any interfaces changed?
NA
- [x] Any backward compatibility impacted?
NA
- [x] Document update required?
YES, it will done in another PR
- [x] Testing done
manual testing is done, UTs will be added in another PR
Please provide details on
- Whether new unit test cases have been added or why no new tests
are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance
test report.
- Any additional information to help reviewers in testing this
change.
- [x] For large changes, please consider breaking it into sub-tasks under
an umbrella JIRA.
NA
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/akashrn5/incubator-carbondata
localdict_prop_desc
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/carbondata/pull/2375.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 #2375
----
commit 5308e1fd3602f4a24d78694b416fda7b6f3a9bcd
Author: akashrn5 <akashnilugal@...>
Date: 2018-06-06T15:03:39Z
Support adding local dictionary configuration in create table statement and
show the configs in describe formatted table
----
---
