GitHub user ajantha-bhat opened a pull request:
https://github.com/apache/carbondata/pull/3055
[HOTFIX] presto can't read dictionary include decimal column
problem: decimal column with dictionary include cannot be read in
presto
cause: int is typecasted to decimal for dictionary columns in decimal
stream reader.
solution: keep original data type as well as new data type for decimal
stream reader.
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:
- [ ] Any interfaces changed? NA
- [ ] Any backward compatibility impacted? NA
- [ ] Document update required? NA
- [ ] Testing done.
done
- [ ] 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/ajantha-bhat/carbondata issue_fix
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/carbondata/pull/3055.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 #3055
----
commit 205189d121f80aa87598a6c5f5e34562036c03c5
Author: ajantha-bhat <ajanthabhat@...>
Date: 2019-01-07T09:20:11Z
dictionary include decimal column type cast issue
problem: decimal column with dictionary include cannot be read in
presto
cause: int is typecasted to decimal for dictionary columns.
solution: keep original data type as well as new data type for decimal
slice reader
----
---
