[GitHub] carbondata pull request #1712: [CARBONDATA-1931]DataLoad failed for Aggregat...

[qiuchenjian-2]

GitHub user BJangir opened a pull request:

    https://github.com/apache/carbondata/pull/1712

    [CARBONDATA-1931]DataLoad failed for Aggregate table when measure is …

    Root Cause  , during data loading ,dataframe columns are sorted based on selection order but  in RowConverter fileds values are compared based on ordinal column .
   
    Solution  = while making CarbonLoadModel set header sorted by ordinal columns instead of selection order.

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

    $ git pull https://github.com/BJangir/incubator-carbondata CARBONDATA-1931

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

    https://github.com/apache/carbondata/pull/1712.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 #1712
   
----
commit f619a97caa7e0e7bd0aaa6326c23055b87a9e99e
Author: BJangir <babulaljangir111@...>
Date:   2017-12-22T08:40:48Z

    [CARBONDATA-1931]DataLoad failed for Aggregate table when measure is used for groupby

----


---

[qiuchenjian-2]

Github user gvramana commented on the issue:

    https://github.com/apache/carbondata/pull/1712
 
    add to whitelist


---

[qiuchenjian-2]

Github user kunal642 commented on the issue:

    https://github.com/apache/carbondata/pull/1712
 
    retest this please


---

[qiuchenjian-2]

Github user gvramana commented on the issue:

    https://github.com/apache/carbondata/pull/1712
 
    LGTM


---

[qiuchenjian-2]

Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/1712
 
    SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/2518/



---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1712
 
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/2267/



---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1712
 
    Build Success with Spark 2.2.0, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/1046/



---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1712
 
    Build Success with Spark 2.2.0, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/1058/



---

[qiuchenjian-2]

Github user kumarvishal09 commented on the issue:

    https://github.com/apache/carbondata/pull/1712
 
    retest this please


---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1712
 
    Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/2314/



---

[qiuchenjian-2]

Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1712
 
    Build Success with Spark 2.2.0, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/1098/



---

[qiuchenjian-2]

Github user BJangir commented on the issue:

    https://github.com/apache/carbondata/pull/1712
 
    Merged


---

[qiuchenjian-2]

Github user BJangir closed the pull request at:

    https://github.com/apache/carbondata/pull/1712


---