GitHub user kumarvishal09 opened a pull request:
https://github.com/apache/carbondata/pull/2815
[WIP]
### What changes are proposed in this PR
**Method In-lining Optimization**
JIT will inline any method if method size is less than 325 byte code size
and if it is called more than 10K times(default value). If method is private or
static it will be easier for JIT to inline as type safe check is not required,
for protected/public method it will add a overhead of type check and because of
this it will not behave as inline.
Because of above case some refactoring is done for primitive no dictionary
data type columns. Earlier ColumnPageWrapper.java was handling query processing
for all primitive no dictionary data type column now in This PR separate
classes are created for each data type handling and all the HOT method is kept
as private and protected methods are overridden and other methods are added in
Super classes
- [ ] Any interfaces changed?
- [ ] Any backward compatibility impacted?
- [ ] Document update required?
- [ ] Testing done
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.
- [ ] For large changes, please consider breaking it into sub-tasks under
an umbrella JIRA.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/kumarvishal09/incubator-carbondata
JITBasedOPtimization
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/carbondata/pull/2815.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 #2815
----
commit f7c8c16610dc332b4385d8970c5aac2272b84623
Author: kumarvishal09 <kumarvishal1802@...>
Date: 2018-10-15T14:32:40Z
PrimitiveDimColumnPage
----
---
