[GitHub] carbondata pull request #937: [CARBONDATA-1076] Join Issue caused by diction...

[qiuchenjian-2]

GitHub user ravipesala opened a pull request:

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

    [CARBONDATA-1076] Join Issue caused by dictionary and shuffle exchange

    When under join aggregate plan is there then there is a high chance of having same partitioning so it will generate single exchange instead of two. So in that case better push down the decoder to aggregate rather than handling at join phase.

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

    $ git pull https://github.com/ravipesala/incubator-carbondata full-join-issue

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

    https://github.com/apache/carbondata/pull/937.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 #937
   
----
commit 006e384c5e4a2aa38c11cc4b7e8ab0118d6d2330
Author: ravipesala <[hidden email]>
Date:   2017-05-23T05:02:21Z

    Fixed full join query issue with aggregate

commit 2c9462772c1bd9bebdd378cc9aad5d51732e56a9
Author: ravipesala <[hidden email]>
Date:   2017-05-23T05:38:44Z

    Fixed in spark-1.6

----


---
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 [hidden email] or file a JIRA ticket
with INFRA.
---

[qiuchenjian-2]

Github user ravipesala commented on the issue:

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


---
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 [hidden email] or file a JIRA ticket
with INFRA.
---

[qiuchenjian-2]

Github user jackylk commented on the issue:

    https://github.com/apache/carbondata/pull/937
 
    LGTM.
    I verified locally. Currently CI has problem


---
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 [hidden email] or file a JIRA ticket
with INFRA.
---

[qiuchenjian-2]

Github user asfgit closed the pull request at:

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


---
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 [hidden email] or file a JIRA ticket
with INFRA.
---