GitHub user ashokblend opened a pull request:

    https://github.com/apache/incubator-carbondata/pull/95

    [CARBONDATA-152]Double min max difference compression issue.

    1.While compressing maxdifference typecast with proper DataType
    2.While writting we follow like
    a. diff with max value , say d = max - x
    b. multiply with 10^decimal i.e w = d*10^decimal
    3.Similarly while reading we should follow like
    a. divide written value 10^decimal i.e d = w/10^decimal
    b. diff with max i.e x = max -d
    4. There is problem in java with double subtraction
    i.e double value = 3.141818 - 0.000610
    java will result 3.1412080000000002
    To avoid this we are converting to BigDecimal and then doing subtraction
    5.Changed logic to calculate no of integer after decimal.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ashokblend/incubator-carbondata 
doubleminmaxcompression

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-carbondata/pull/95.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 #95
    
----
commit 6c1c959a2c75d4e1d297f8ce2ac9056cdacf44c0
Author: Ashok Kumar <[email protected]>
Date:   2016-08-25T10:40:29Z

    Double min max difference compression issue.
    1.While compressing maxdifference typecast with proper DataType
    2.While writting we follow like
    a. diff with max value , say d = max - x
    b. multiply with 10^decimal i.e w = d*10^decimal
    3.Similarly while reading we should follow like
    a. divide written value 10^decimal i.e d = w/10^decimal
    b. diff with max i.e x = max -d
    4. There is problem in java with double subtraction
    i.e double value = 3.141818 - 0.000610
    java will result 3.1412080000000002
    To avoid this we are converting to BigDecimal and then doing subtraction
    5.Changed logic to calculate no of integer after decimal.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to