[GitHub] incubator-carbondata issue #635: [WIP]support SORT_COLUMNS

2017-03-15 Thread QiangCai
Github user QiangCai commented on the issue: https://github.com/apache/incubator-carbondata/pull/635 @kumarvishal09 1. If user has not mentioned any sort column then it will go to old flow, sorting based on all dimension column 2. yes 3. During dataloading, the start/end

[GitHub] incubator-carbondata issue #635: [WIP]support SORT_COLUMNS

2017-03-15 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/incubator-carbondata/pull/635 Build Success with Spark 1.6.2, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1174/ --- If your project is set up for it, you can reply to this email and

[GitHub] incubator-carbondata issue #635: [WIP]support SORT_COLUMNS

2017-03-15 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/incubator-carbondata/pull/635 Build Success with Spark 1.6.2, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1172/ --- If your project is set up for it, you can reply to this email and

[GitHub] incubator-carbondata issue #635: [WIP]support SORT_COLUMNS

2017-03-15 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/incubator-carbondata/pull/635 Build Failed with Spark 1.6.2, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1171/ --- If your project is set up for it, you can reply to this email and

[GitHub] incubator-carbondata issue #635: [WIP]support SORT_COLUMNS

2017-03-14 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/incubator-carbondata/pull/635 Build Success with Spark 1.6.2, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1140/ --- If your project is set up for it, you can reply to this email and

[GitHub] incubator-carbondata issue #635: [WIP]support SORT_COLUMNS

2017-03-10 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/incubator-carbondata/pull/635 Build Failed with Spark 1.6.2, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1078/ --- If your project is set up for it, you can reply to this email and

[GitHub] incubator-carbondata issue #635: [WIP]support SORT_COLUMNS

2017-03-09 Thread kumarvishal09
Github user kumarvishal09 commented on the issue: https://github.com/apache/incubator-carbondata/pull/635 @QiangCai I have queries related to this PR. 1. If user has not mentioned any sort column then it will go to old flow (sorting based on all dimension column) or data wont be

[GitHub] incubator-carbondata issue #635: [WIP]support SORT_COLUMNS

2017-03-08 Thread QiangCai
Github user QiangCai commented on the issue: https://github.com/apache/incubator-carbondata/pull/635 @ravipesala Is it neccessary to limit that the sort_columns should come from dimensions? If the table need be sorted by a measure, we should use dictionary_include to add it

[GitHub] incubator-carbondata issue #635: [WIP]support SORT_COLUMNS

2017-03-08 Thread QiangCai
Github user QiangCai commented on the issue: https://github.com/apache/incubator-carbondata/pull/635 @ravipesala I have listed the tasks. Better to implement another direct-dictionary encoding for numeric datatype column. We can remove the dimension and measure concept, and only

[GitHub] incubator-carbondata issue #635: [WIP]support SORT_COLUMNS

2017-03-08 Thread ravipesala
Github user ravipesala commented on the issue: https://github.com/apache/incubator-carbondata/pull/635 @QiangCai Please mention what are the tasks you are doing in this PR. It is better to stick only supporting sort_columns in this PR. Other tasks can be pushed to other PRs. --- If

[GitHub] incubator-carbondata issue #635: [WIP]support SORT_COLUMNS

2017-03-08 Thread QiangCai
Github user QiangCai commented on the issue: https://github.com/apache/incubator-carbondata/pull/635 @ravipesala good suggestion. Direct dicitonary is better than no dictioanry. I will add it. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] incubator-carbondata issue #635: [WIP]support SORT_COLUMNS

2017-03-08 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/incubator-carbondata/pull/635 Build Success with Spark 1.6.2, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1047/ --- If your project is set up for it, you can reply to this email and

[GitHub] incubator-carbondata issue #635: [WIP]support SORT_COLUMNS

2017-03-08 Thread ravipesala
Github user ravipesala commented on the issue: https://github.com/apache/incubator-carbondata/pull/635 @QiangCai I have few doubts. Why primitive data types are supported as no-dictionary columns in this PR? It supposed to be direct dictionary. Why date and timestamp are

[GitHub] incubator-carbondata issue #635: [WIP]support SORT_COLUMNS

2017-03-08 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/incubator-carbondata/pull/635 Build Failed with Spark 1.6.2, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1041/ --- If your project is set up for it, you can reply to this email and