[GitHub] incubator-carbondata pull request #208: [CARBONDATA-284][WIP] Abstracting in...

2016-10-07 Thread JihongMA
Github user JihongMA commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/208#discussion_r82487685 --- Diff: hadoop/src/main/java/org/apache/carbondata/hadoop/api/columnar/CarbonColumnarInputFormat.java --- @@ -0,0 +1,56 @@ +/* + *

[GitHub] incubator-carbondata pull request #208: [CARBONDATA-284][WIP] Abstracting in...

2016-10-07 Thread JihongMA
Github user JihongMA commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/208#discussion_r82487219 --- Diff: hadoop/src/main/java/org/apache/carbondata/hadoop/api/CarbonInputFormatBase.java --- @@ -0,0 +1,69 @@ +/* --- End diff --

Re: [VOTE] Apache CarbonData 0.1.1-incubating release

2016-10-07 Thread Gangumalla, Uma
+1 (binding) Thanks for making the release. Build is fine DISCLAIMER exists. It has Œincubating¹ tag NOTICE & LICENCE files exists Regards, Uma On 10/5/16, 3:06 AM, "Liang Big data" wrote: >Hi all, > >PPMC vote for the release Apache CarbonData 0.1.1-incubating has

[GitHub] incubator-carbondata pull request #208: [CARBONDATA-284][WIP] Abstracting in...

2016-10-07 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/208#discussion_r82446169 --- Diff: hadoop/src/main/java/org/apache/carbondata/hadoop/api/row/CarbonRowInputFormat.java --- @@ -0,0 +1,40 @@ +/* + * Licensed

Re: [VOTE] Apache CarbonData 0.1.1-incubating release

2016-10-07 Thread Henry Saputra
DISCLAIMER file exits LICENSE file looks good NOTICE file look good Artifacts has incubating tag No executables binaries in source artifacts Can compile from source +1 (binding) - Henry On Wed, Oct 5, 2016 at 3:06 AM, Liang Big data wrote: > Hi all, > > PPMC vote for

[GitHub] incubator-carbondata pull request #215: [WIP][CARBONDATA-2] Remove kettle fr...

2016-10-07 Thread ravipesala
GitHub user ravipesala opened a pull request: https://github.com/apache/incubator-carbondata/pull/215 [WIP][CARBONDATA-2] Remove kettle from carbodata New flow is added in processing module to load data with out kettle. New code is added to the package

[GitHub] incubator-carbondata pull request #203: [CARBONDATA-279]Load RDD to carbonda...

2016-10-07 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/203#discussion_r82413334 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/CarbonOption.scala --- @@ -17,6 +17,8 @@ package

[GitHub] incubator-carbondata pull request #203: [CARBONDATA-279]Load RDD to carbonda...

2016-10-07 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/203#discussion_r82413317 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataLoadRDD.scala --- @@ -72,8 +76,91 @@ class

[GitHub] incubator-carbondata pull request #203: [CARBONDATA-279]Load RDD to carbonda...

2016-10-07 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/203#discussion_r82413331 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/CarbonOption.scala --- @@ -36,6 +38,8 @@ class CarbonOption(options:

[GitHub] incubator-carbondata pull request #203: [CARBONDATA-279]Load RDD to carbonda...

2016-10-07 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/203#discussion_r82413313 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataLoadRDD.scala --- @@ -462,3 +478,155 @@ class

[GitHub] incubator-carbondata pull request #204: [CARBONDATA-280]Fix the bug that whe...

2016-10-07 Thread Zhangshunyu
Github user Zhangshunyu closed the pull request at: https://github.com/apache/incubator-carbondata/pull/204 --- 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

[GitHub] incubator-carbondata pull request #203: [CARBONDATA-279]Load RDD to carbonda...

2016-10-07 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/203#discussion_r82387284 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/util/GlobalDictionaryUtil.scala --- @@ -742,7 +742,8 @@ object

[GitHub] incubator-carbondata pull request #203: [CARBONDATA-279]Load RDD to carbonda...

2016-10-07 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/203#discussion_r82369947 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataLoadRDD.scala --- @@ -72,8 +76,91 @@ class

[GitHub] incubator-carbondata pull request #203: [CARBONDATA-279]Load RDD to carbonda...

2016-10-07 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/203#discussion_r82370864 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataLoadRDD.scala --- @@ -462,3 +478,155 @@ class

[GitHub] incubator-carbondata pull request #203: [CARBONDATA-279]Load RDD to carbonda...

2016-10-07 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/203#discussion_r82367638 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/CarbonOption.scala --- @@ -17,6 +17,8 @@ package

[GitHub] incubator-carbondata pull request #203: [CARBONDATA-279]Load RDD to carbonda...

2016-10-07 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/203#discussion_r82363963 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/CarbonOption.scala --- @@ -36,6 +38,8 @@ class CarbonOption(options:

[GitHub] incubator-carbondata pull request #148: [CARBONDATA-233] bad record logger s...

2016-10-07 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/148#discussion_r82348806 --- Diff: processing/src/main/java/org/apache/carbondata/processing/surrogatekeysgenerator/csvbased/CarbonCSVBasedSeqGenStep.java --- @@

[GitHub] incubator-carbondata pull request #203: [CARBONDATA-279]Load RDD to carbonda...

2016-10-07 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/203#discussion_r82344172 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataLoadRDD.scala --- @@ -462,3 +478,133 @@ class

[GitHub] incubator-carbondata pull request #148: [CARBONDATA-233] bad record logger s...

2016-10-07 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/148#discussion_r82341356 --- Diff: processing/src/main/java/org/apache/carbondata/processing/surrogatekeysgenerator/csvbased/BadRecordslogger.java --- @@ -89,41

[GitHub] incubator-carbondata pull request #148: [CARBONDATA-233] bad record logger s...

2016-10-07 Thread sujith71955
Github user sujith71955 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/148#discussion_r82339778 --- Diff: processing/src/main/java/org/apache/carbondata/processing/surrogatekeysgenerator/csvbased/BadRecordslogger.java --- @@ -89,41

[GitHub] incubator-carbondata pull request #203: [CARBONDATA-279]Load RDD to carbonda...

2016-10-07 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/203#discussion_r82334174 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataLoadRDD.scala --- @@ -72,6 +76,89 @@ class

[GitHub] incubator-carbondata pull request #203: [CARBONDATA-279]Load RDD to carbonda...

2016-10-07 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/203#discussion_r82334120 --- Diff: processing/src/main/java/org/apache/carbondata/processing/csvreaderstep/CsvInput.java --- @@ -343,41 +349,57 @@ public boolean

[GitHub] incubator-carbondata pull request #203: [CARBONDATA-279]Load RDD to carbonda...

2016-10-07 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/203#discussion_r82333906 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/CarbonOption.scala --- @@ -17,13 +17,15 @@ package