GitHub user dhatchayani opened a pull request:
https://github.com/apache/carbondata/pull/1374
[CARBONDATA-1491] Dictionary_exclude columns are not going into
no_dictionary flow
(1) **DICTIONARY_EXCLUDE** columns are not considered as no_dictionary
columns - while parsing we are not setting nodictionary columns.
(2) For reconstructing defaultValue for newly added no dictionary measure
column, logic is changed, as the previous logic can throw
IllegalArgumentException for wrong length.
(3) Test cases are added for the same.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/dhatchayani/incubator-carbondata
add_dic_excl_measure
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/carbondata/pull/1374.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 #1374
----
commit 4a812e70f3f2aa023e1fbf509f287766a14d8521
Author: dhatchayani <[email protected]>
Date: 2017-09-20T06:33:18Z
[CARBONDATA-1491] Dictionary_exclude columns are not going into
no_dictionary flow
----
---
