[GitHub] carbondata issue #2940: [CARBONDATA-3116] Support set carbon.query.directQue...

2018-12-02 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/2940 Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1616/ ---

[GitHub] carbondata pull request #2940: [CARBONDATA-3116] Support set carbon.query.di...

2018-12-02 Thread xubo245
Github user xubo245 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2940#discussion_r238166894 --- Diff: integration/spark-common-test/src/test/scala/org/apache/carbondata/integration/spark/testsuite/preaggregate/TestPreAggCreateCommand.scala ---

[GitHub] carbondata issue #2970: [CARBONDATA-3142]Add timestamp with thread name whic...

2018-12-02 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/2970 Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1615/ ---

[GitHub] carbondata pull request #2970: [CARBONDATA-3142]Add timestamp with thread na...

2018-12-02 Thread qiuchenjian
GitHub user qiuchenjian opened a pull request: https://github.com/apache/carbondata/pull/2970 [CARBONDATA-3142]Add timestamp with thread name which created by CarbonThreadFactory [CARBONDATA-3142]Add timestamp with thread name which created by CarbonThreadFactory Be sure

[GitHub] carbondata pull request #2940: [CARBONDATA-3116] Support set carbon.query.di...

2018-12-02 Thread kunal642
Github user kunal642 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2940#discussion_r238161863 --- Diff: integration/spark2/pom.xml --- @@ -105,6 +105,11 @@ + + org.apache.httpcomponents

[GitHub] carbondata pull request #2821: [CARBONDATA-3017] Map DDL Support

2018-12-02 Thread Indhumathi27
Github user Indhumathi27 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2821#discussion_r238161031 --- Diff: integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/createTable/TestCreateDDLForComplexMapType.scala ---

[GitHub] carbondata pull request #2821: [CARBONDATA-3017] Map DDL Support

2018-12-02 Thread Indhumathi27
Github user Indhumathi27 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2821#discussion_r238160123 --- Diff: integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/createTable/TestCreateDDLForComplexMapType.scala ---

[GitHub] carbondata issue #2878: [CARBONDATA-3107] Optimize error/exception coding fo...

2018-12-02 Thread kevinjmh
Github user kevinjmh commented on the issue: https://github.com/apache/carbondata/pull/2878 conflicts fixed. ---

[jira] [Created] (CARBONDATA-3142) The names of threads created by CarbonThreadFactory are all the same

2018-12-02 Thread Chenjian Qiu (JIRA)
Chenjian Qiu created CARBONDATA-3142: Summary: The names of threads created by CarbonThreadFactory are all the same Key: CARBONDATA-3142 URL: https://issues.apache.org/jira/browse/CARBONDATA-3142

[GitHub] carbondata pull request #2940: [CARBONDATA-3116] Support set carbon.query.di...

2018-12-02 Thread xubo245
Github user xubo245 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2940#discussion_r238156400 --- Diff: integration/spark-common-test/src/test/scala/org/apache/carbondata/integration/spark/testsuite/preaggregate/TestPreAggCreateCommand.scala ---

[GitHub] carbondata pull request #2963: [CARBONDATA-3139] Fix bugs in MinMaxDataMap e...

2018-12-02 Thread Indhumathi27
Github user Indhumathi27 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2963#discussion_r238156188 --- Diff: integration/spark2/src/test/scala/org/apache/carbondata/datamap/minmax/MinMaxDataMapFunctionSuite.scala --- @@ -0,0 +1,415 @@ +/*

[GitHub] carbondata issue #2931: [CARBONDATA-2999] support read schema from S3

2018-12-02 Thread xubo245
Github user xubo245 commented on the issue: https://github.com/apache/carbondata/pull/2931 @KanakaKumar @jackylk @QiangCai @ajantha-bhat @kunal642 Rebased, Please review it. ---

[GitHub] carbondata pull request #2969: [CARBONDATA-3127]Fix the TestCarbonSerde exce...

2018-12-02 Thread xubo245
Github user xubo245 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2969#discussion_r238154723 --- Diff: integration/hive/src/test/java/org/apache/carbondata/hive/TestCarbonSerDe.java --- @@ -0,0 +1,133 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #2963: [CARBONDATA-3139] Fix bugs in MinMaxDataMap e...

2018-12-02 Thread dhatchayani
Github user dhatchayani commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2963#discussion_r238154502 --- Diff: datamap/example/src/main/java/org/apache/carbondata/datamap/minmax/MinMaxDataMapFactory.java --- @@ -0,0 +1,365 @@ +/* + *

[GitHub] carbondata issue #2969: [CARBONDATA-3127]Fix the TestCarbonSerde exception

2018-12-02 Thread xubo245
Github user xubo245 commented on the issue: https://github.com/apache/carbondata/pull/2969 LGTM ---

[GitHub] carbondata pull request #2963: [CARBONDATA-3139] Fix bugs in MinMaxDataMap e...

2018-12-02 Thread dhatchayani
Github user dhatchayani commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2963#discussion_r238153927 --- Diff: datamap/example/src/main/java/org/apache/carbondata/datamap/minmax/MinMaxDataMapFactory.java --- @@ -0,0 +1,365 @@ +/* + *

[GitHub] carbondata pull request #2963: [CARBONDATA-3139] Fix bugs in MinMaxDataMap e...

2018-12-02 Thread dhatchayani
Github user dhatchayani commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2963#discussion_r238153188 --- Diff: datamap/example/src/main/java/org/apache/carbondata/datamap/minmax/AbstractMinMaxDataMapWriter.java --- @@ -0,0 +1,248 @@ +/* +

[GitHub] carbondata pull request #2963: [CARBONDATA-3139] Fix bugs in MinMaxDataMap e...

2018-12-02 Thread Indhumathi27
Github user Indhumathi27 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2963#discussion_r238152299 --- Diff: datamap/example/src/main/java/org/apache/carbondata/datamap/minmax/AbstractMinMaxDataMapWriter.java --- @@ -0,0 +1,248 @@ +/* +

[GitHub] carbondata pull request #2940: [CARBONDATA-3116] Support set carbon.query.di...

2018-12-02 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2940#discussion_r238152763 --- Diff: integration/spark-common-test/src/test/scala/org/apache/carbondata/integration/spark/testsuite/preaggregate/TestPreAggCreateCommand.scala

[GitHub] carbondata issue #2931: [CARBONDATA-2999] support read schema from S3

2018-12-02 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/2931 Build Success with Spark 2.2.1, Please check CI http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1825/ ---

[GitHub] carbondata issue #2931: [CARBONDATA-2999] support read schema from S3

2018-12-02 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/2931 Build Success with Spark 2.3.1, Please check CI http://136.243.101.176:8080/job/carbondataprbuilder2.3/9874/ ---

[GitHub] carbondata pull request #2940: [CARBONDATA-3116] Support set carbon.query.di...

2018-12-02 Thread kunal642
Github user kunal642 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2940#discussion_r238142863 --- Diff: integration/spark-common-test/src/test/scala/org/apache/carbondata/integration/spark/testsuite/preaggregate/TestPreAggCreateCommand.scala ---

[GitHub] carbondata issue #2931: [CARBONDATA-2999] support read schema from S3

2018-12-02 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/2931 Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1614/ ---

[GitHub] carbondata issue #2969: [CARBONDATA-3127]Fix the TestCarbonSerde exception

2018-12-02 Thread zzcclp
Github user zzcclp commented on the issue: https://github.com/apache/carbondata/pull/2969 LGTM ---

[GitHub] carbondata issue #2940: [CARBONDATA-3116] Support set carbon.query.directQue...

2018-12-02 Thread xubo245
Github user xubo245 commented on the issue: https://github.com/apache/carbondata/pull/2940 @jackylk @ravipesala @KanakaKumar @kunal642 This is a bug, please review it. ---

[GitHub] carbondata issue #2946: [CARBONDATA-3094] Support concurrent read carbondata...

2018-12-02 Thread xubo245
Github user xubo245 commented on the issue: https://github.com/apache/carbondata/pull/2946 @KanakaKumar @ravipesala @jackylk @kunal642 @ajantha-bhat CI pass, Please review it. ---

[GitHub] carbondata issue #2878: [CARBONDATA-3107] Optimize error/exception coding fo...

2018-12-02 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/2878 Build Success with Spark 2.3.1, Please check CI http://136.243.101.176:8080/job/carbondataprbuilder2.3/9873/ ---

[GitHub] carbondata issue #2878: [CARBONDATA-3107] Optimize error/exception coding fo...

2018-12-02 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/2878 Build Success with Spark 2.2.1, Please check CI http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1824/ ---

[GitHub] carbondata issue #2878: [CARBONDATA-3107] Optimize error/exception coding fo...

2018-12-02 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/2878 Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1613/ ---

[GitHub] carbondata issue #2878: [CARBONDATA-3107] Optimize error/exception coding fo...

2018-12-02 Thread xuchuanyin
Github user xuchuanyin commented on the issue: https://github.com/apache/carbondata/pull/2878 LGTM Please fix the conflicts ---

[GitHub] carbondata issue #2966: [WIP] test and check no sort by default

2018-12-02 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/2966 Build Failed with Spark 2.3.1, Please check CI http://136.243.101.176:8080/job/carbondataprbuilder2.3/9872/ ---

[GitHub] carbondata issue #2966: [WIP] test and check no sort by default

2018-12-02 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/2966 Build Failed with Spark 2.2.1, Please check CI http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1823/ ---

[GitHub] carbondata issue #2966: [WIP] test and check no sort by default

2018-12-02 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/2966 Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1612/ ---

[GitHub] carbondata issue #2966: [WIP] test and check no sort by default

2018-12-02 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/2966 Build Failed with Spark 2.2.1, Please check CI http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1822/ ---

[GitHub] carbondata issue #2966: [WIP] test and check no sort by default

2018-12-02 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/2966 Build Failed with Spark 2.3.1, Please check CI http://136.243.101.176:8080/job/carbondataprbuilder2.3/9871/ ---

[GitHub] carbondata issue #2966: [WIP] test and check no sort by default

2018-12-02 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/2966 Build Failed with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1611/ ---

[GitHub] carbondata pull request #2969: [CARBONDATA-3127]Fix the TestCarbonSerde exce...

2018-12-02 Thread SteNicholas
Github user SteNicholas commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2969#discussion_r238089149 --- Diff: integration/hive/src/test/java/org/apache/carbondata/hive/TestCarbonSerde.java --- @@ -1,133 +1,133 @@ -///* -// * Licensed to

[GitHub] carbondata pull request #2969: [CARBONDATA-3127]Fix the TestCarbonSerde exce...

2018-12-02 Thread SteNicholas
Github user SteNicholas commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2969#discussion_r238089137 --- Diff: integration/hive/src/test/java/org/apache/carbondata/hive/TestCarbonSerde.java --- @@ -1,133 +1,133 @@ -///* -// * Licensed to

[GitHub] carbondata pull request #2969: [CARBONDATA-3127]Fix the TestCarbonSerde exce...

2018-12-02 Thread SteNicholas
Github user SteNicholas commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2969#discussion_r238089109 --- Diff: integration/hive/src/test/java/org/apache/carbondata/hive/TestCarbonSerde.java --- @@ -1,133 +1,133 @@ -///* -// * Licensed to