[GitHub] incubator-carbondata pull request #265: [CARBONDATA-458]Improving First time...

2016-11-30 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/265#discussion_r90250147 --- Diff: core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java --- @@ -814,85 +810,86 @@ * Rocord size

[GitHub] incubator-carbondata issue #372: [CARBONDATA-473] integrate with spark 2 sta...

2016-11-30 Thread jackylk
Github user jackylk commented on the issue: https://github.com/apache/incubator-carbondata/pull/372 CI running http://136.243.101.176:8080/job/ApacheCarbonManualPRBuilder/716/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-carbondata issue #372: [CARBONDATA-473] integrate with spark 2 sta...

2016-11-30 Thread jackylk
Github user jackylk commented on the issue: https://github.com/apache/incubator-carbondata/pull/372 CI passed http://136.243.101.176:8080/job/ApacheCarbonManualPRBuilder/717/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-carbondata issue #265: [CARBONDATA-458]Improving First time query ...

2016-11-30 Thread jackylk
Github user jackylk commented on the issue: https://github.com/apache/incubator-carbondata/pull/265 please add testcase for V2 write and read --- 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

[GitHub] incubator-carbondata issue #372: [CARBONDATA-473] integrate with spark 2 sta...

2016-11-30 Thread jackylk
Github user jackylk commented on the issue: https://github.com/apache/incubator-carbondata/pull/372 CI running http://136.243.101.176:8080/job/ApacheCarbonManualPRBuilder/715/ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] incubator-carbondata pull request #372: [CARBONDATA-473] integrate with spar...

2016-11-30 Thread scwf
GitHub user scwf opened a pull request: https://github.com/apache/incubator-carbondata/pull/372 [CARBONDATA-473] integrate with spark 2 stable datasource api 1. read path: based on spark2 datasource api 2. load path: give a tool to do data load You can merge this pull

[GitHub] incubator-carbondata pull request #368: [CARBONDATA-465] Spark streaming dat...

2016-11-30 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/368#discussion_r90195041 --- Diff: integration/spark/src/main/scala/org/apache/spark/sql/hive/CarbonMetastoreCatalog.scala --- @@ -307,10 +307,17 @@ class

[GitHub] incubator-carbondata pull request #368: [CARBONDATA-465] Spark streaming dat...

2016-11-30 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/368#discussion_r90197023 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonDataProcessorUtil.java --- @@ -604,4 +606,33 @@ public

[GitHub] incubator-carbondata pull request #368: [CARBONDATA-465] Spark streaming dat...

2016-11-30 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/368#discussion_r90194407 --- Diff: integration/spark/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchema.scala --- @@ -763,10 +763,9 @@ case

[GitHub] incubator-carbondata issue #372: [CARBONDATA-473] integrate with spark 2 sta...

2016-11-30 Thread jackylk
Github user jackylk commented on the issue: https://github.com/apache/incubator-carbondata/pull/372 Is there an example to run carbon with spark2? Can these code work? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] incubator-carbondata pull request #372: [CARBONDATA-473] integrate with spar...

2016-11-30 Thread scwf
Github user scwf commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/372#discussion_r90207315 --- Diff: integration/spark-common/pom.xml --- @@ -84,6 +79,11 @@ junit + com.databricks +

[GitHub] incubator-carbondata issue #362: [CARBONDATA-459] Block distribution is wron...

2016-11-30 Thread manishgupta88
Github user manishgupta88 commented on the issue: https://github.com/apache/incubator-carbondata/pull/362 @jackylk ...Handled all review comments. Kindly review and merge --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[jira] [Created] (CARBONDATA-475) Implement unit test cases for core.carbon.querystatics package

2016-11-30 Thread Anurag Srivastava (JIRA)
Anurag Srivastava created CARBONDATA-475: Summary: Implement unit test cases for core.carbon.querystatics package Key: CARBONDATA-475 URL: https://issues.apache.org/jira/browse/CARBONDATA-475

[jira] [Created] (CARBONDATA-474) Implement unit test cases for core.datastorage.store.columnar package

2016-11-30 Thread Anurag Srivastava (JIRA)
Anurag Srivastava created CARBONDATA-474: Summary: Implement unit test cases for core.datastorage.store.columnar package Key: CARBONDATA-474 URL: https://issues.apache.org/jira/browse/CARBONDATA-474

[GitHub] incubator-carbondata pull request #368: [CARBONDATA-465] Spark streaming dat...

2016-11-30 Thread allwefantasy
Github user allwefantasy commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/368#discussion_r90229415 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonDataProcessorUtil.java --- @@ -604,4 +606,33 @@ public

[GitHub] incubator-carbondata pull request #374: [Carbondata 475] Added test cases fo...

2016-11-30 Thread anubhav100
GitHub user anubhav100 opened a pull request: https://github.com/apache/incubator-carbondata/pull/374 [Carbondata 475] Added test cases for org.apache.carbondata.core.carbon.querystatistics Be sure to do all of the following to help us incorporate your contribution quickly and

[GitHub] incubator-carbondata issue #372: [CARBONDATA-473] integrate with spark 2 sta...

2016-11-30 Thread scwf
Github user scwf commented on the issue: https://github.com/apache/incubator-carbondata/pull/372 the result has some issues for the carbon example: ``` +--++---+---+---+ |shortField|intField|bigintField|doubleField|stringField|

[GitHub] incubator-carbondata issue #372: [CARBONDATA-473] integrate with spark 2 sta...

2016-11-30 Thread jackylk
Github user jackylk commented on the issue: https://github.com/apache/incubator-carbondata/pull/372 Yes, bigint datatype has some issue, please raise a JIRA for it. And can you add some basic testcase for this PR --- If your project is set up for it, you can reply to this email

[GitHub] incubator-carbondata pull request #375: Fix compile error for spark1.6 integ...

2016-11-30 Thread jackylk
GitHub user jackylk opened a pull request: https://github.com/apache/incubator-carbondata/pull/375 Fix compile error for spark1.6 integration You can merge this pull request into a Git repository by running: $ git pull https://github.com/jackylk/incubator-carbondata spark16

[GitHub] incubator-carbondata pull request #362: [CARBONDATA-459] Block distribution ...

2016-11-30 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/362#discussion_r90365594 --- Diff: integration/spark/src/main/scala/org/apache/spark/sql/hive/DistributionUtil.scala --- @@ -101,47 +101,107 @@ object DistributionUtil

[GitHub] incubator-carbondata pull request #338: [CARBONDATA-100]Implement BigInt val...

2016-11-30 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/338#discussion_r90394339 --- Diff: core/src/main/java/org/apache/carbondata/core/compression/BigIntCompressor.java --- @@ -0,0 +1,134 @@ +/* + * Licensed to

[GitHub] incubator-carbondata pull request #265: [CARBONDATA-458]Improving First time...

2016-11-30 Thread kumarvishal09
Github user kumarvishal09 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/265#discussion_r90395244 --- Diff: core/src/main/java/org/apache/carbondata/core/carbon/datastore/chunk/reader/measure/AbstractMeasureChunkReader.java --- @@

[GitHub] incubator-carbondata pull request #265: [CARBONDATA-458]Improving First time...

2016-11-30 Thread kumarvishal09
Github user kumarvishal09 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/265#discussion_r90395320 --- Diff: core/src/main/java/org/apache/carbondata/core/carbon/metadata/blocklet/DataFileFooter.java --- @@ -38,7 +38,7 @@ /**

[GitHub] incubator-carbondata pull request #338: [CARBONDATA-100]Implement BigInt val...

2016-11-30 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/338#discussion_r90397726 --- Diff: core/src/main/java/org/apache/carbondata/core/compression/DoubleCompressor.java --- @@ -0,0 +1,307 @@ +/* + * Licensed to

[GitHub] incubator-carbondata pull request #265: [CARBONDATA-458]Improving First time...

2016-11-30 Thread kumarvishal09
Github user kumarvishal09 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/265#discussion_r90386030 --- Diff: core/src/main/java/org/apache/carbondata/core/carbon/datastore/block/TableBlockInfo.java --- @@ -81,13 +84,14 @@ public

[GitHub] incubator-carbondata pull request #265: [CARBONDATA-458]Improving First time...

2016-11-30 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/265#discussion_r90244822 --- Diff: core/src/main/java/org/apache/carbondata/core/carbon/datastore/block/TableBlockInfo.java --- @@ -81,13 +84,14 @@ public

[GitHub] incubator-carbondata pull request #265: [CARBONDATA-458]Improving First time...

2016-11-30 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/265#discussion_r90245110 --- Diff: core/src/main/java/org/apache/carbondata/core/carbon/datastore/chunk/reader/CarbonDataReaderFactory.java --- @@ -0,0 +1,93 @@

[GitHub] incubator-carbondata pull request #265: [CARBONDATA-458]Improving First time...

2016-11-30 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/265#discussion_r90249003 --- Diff: core/src/main/java/org/apache/carbondata/core/carbon/datastore/chunk/reader/measure/AbstractMeasureChunkReader.java --- @@ -18,58

[GitHub] incubator-carbondata pull request #265: [CARBONDATA-458]Improving First time...

2016-11-30 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/265#discussion_r90249260 --- Diff: core/src/main/java/org/apache/carbondata/core/carbon/datastore/chunk/reader/measure/CompressedMeasureChunkFileReader2.java --- @@

[GitHub] incubator-carbondata pull request #265: [CARBONDATA-458]Improving First time...

2016-11-30 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/265#discussion_r90249468 --- Diff: core/src/main/java/org/apache/carbondata/core/carbon/metadata/blocklet/DataFileFooter.java --- @@ -38,7 +38,7 @@ /**

[jira] [Created] (CARBONDATA-476) storeLocation start with file:/// cause table not found exceptioin

2016-11-30 Thread WilliamZhu (JIRA)
WilliamZhu created CARBONDATA-476: - Summary: storeLocation start with file:/// cause table not found exceptioin Key: CARBONDATA-476 URL: https://issues.apache.org/jira/browse/CARBONDATA-476 Project:

[jira] [Created] (CARBONDATA-477) Can we merge storeLocation and metaLocation into one?

2016-11-30 Thread WilliamZhu (JIRA)
WilliamZhu created CARBONDATA-477: - Summary: Can we merge storeLocation and metaLocation into one? Key: CARBONDATA-477 URL: https://issues.apache.org/jira/browse/CARBONDATA-477 Project: CarbonData