[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119528515 --- Diff: core/src/main/java/org/apache/carbondata/core/datastore/impl/array/DataMapStoreFactory.java --- @@ -0,0 +1,93 @@ +/* + * Licensed

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119528455 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/schema/IndexSchema.java --- @@ -0,0 +1,37 @@ +/* + * Licensed to the

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119528159 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/DataMap.java --- @@ -0,0 +1,105 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119528141 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/DataMap.java --- @@ -0,0 +1,105 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119527990 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/DataMap.java --- @@ -0,0 +1,105 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119527950 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/DataMap.java --- @@ -0,0 +1,105 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119527882 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/schema/DataMapSchemaType.java --- @@ -0,0 +1,24 @@ +/* + * Licensed to

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119527857 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/schema/DataMapSchemaType.java --- @@ -0,0 +1,24 @@ +/* + * Licensed to

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119527699 --- Diff: core/src/main/java/org/apache/carbondata/core/datastore/block/TableBlockInfo.java --- @@ -307,4 +316,45 @@ public void

[GitHub] carbondata pull request #981: [CARBONDATA-1111]Improve No dictionary column ...

2017-05-31 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/981#discussion_r119526605 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/filter/executer/ExcludeFilterExecuterImpl.java --- @@ -35,14 +33,19 @@

[GitHub] carbondata pull request #984: [CARBONDATA-1008] Make Caron table schema comp...

2017-05-31 Thread anubhav100
Github user anubhav100 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/984#discussion_r119525572 --- Diff: integration/hive/src/main/scala/org/apache/carbondata/hiveexample/HiveExample.scala --- @@ -54,15 +54,15 @@ object HiveExample {

[GitHub] carbondata issue #979: [CARBONDATA-1102] resolved int,short type bug for hiv...

2017-05-31 Thread anubhav100
Github user anubhav100 commented on the issue: https://github.com/apache/carbondata/pull/979 @chenliang613 both build are successfull now can you merge it --- 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

[GitHub] carbondata pull request #986: [CARBONDATA-1108] Supported IUD for vector rea...

2017-05-31 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/986#discussion_r119524856 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/collector/impl/DictionaryBasedVectorResultCollector.java --- @@ -144,6 +144,8 @@ protected

[GitHub] carbondata issue #979: [CARBONDATA-1102] resolved int,short type bug for hiv...

2017-05-31 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/979 apache jenkins CI are not stable, let me check it at my local machine also. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] carbondata issue #979: [CARBONDATA-1102] resolved int,short type bug for hiv...

2017-05-31 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/979 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

[GitHub] carbondata issue #978: [CARBONDATA-1109] Cover the case when last page is no...

2017-05-31 Thread watermen
Github user watermen commented on the issue: https://github.com/apache/carbondata/pull/978 @kumarvishal09 It is very hard to write the testcase because this case is happened occasionally. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] carbondata issue #978: [CARBONDATA-1109] Cover the case when last page is no...

2017-05-31 Thread watermen
Github user watermen commented on the issue: https://github.com/apache/carbondata/pull/978 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

[GitHub] carbondata issue #978: [CARBONDATA-1109] Cover the case when last page is no...

2017-05-31 Thread watermen
Github user watermen commented on the issue: https://github.com/apache/carbondata/pull/978 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

[GitHub] carbondata issue #978: [CARBONDATA-1109] Cover the case when last page is no...

2017-05-31 Thread watermen
Github user watermen commented on the issue: https://github.com/apache/carbondata/pull/978 test it 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,

[GitHub] carbondata issue #981: [CARBONDATA-1111]Improve No dictionary column Include...

2017-05-31 Thread kumarvishal09
Github user kumarvishal09 commented on the issue: https://github.com/apache/carbondata/pull/981 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

[GitHub] carbondata pull request #972: [WIP] Added set command in carbon to update pr...

2017-05-31 Thread mohammadshahidkhan
Github user mohammadshahidkhan commented on a diff in the pull request: https://github.com/apache/carbondata/pull/972#discussion_r119401097 --- Diff: core/src/main/java/org/apache/carbondata/core/util/SessionParams.java --- @@ -0,0 +1,148 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #972: [WIP] Added set command in carbon to update pr...

2017-05-31 Thread mohammadshahidkhan
Github user mohammadshahidkhan commented on a diff in the pull request: https://github.com/apache/carbondata/pull/972#discussion_r119400763 --- Diff: core/src/main/java/org/apache/carbondata/core/util/SessionParams.java --- @@ -0,0 +1,148 @@ +/* + * Licensed to the Apache

[GitHub] carbondata issue #909: [CARBONDATA-1049] avoid logging raw data into driver ...

2017-05-31 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/909 Build Failed with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/2130/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] carbondata issue #982: [CARBONDATA-1112] fix failing IUD testcase

2017-05-31 Thread ravipesala
Github user ravipesala commented on the issue: https://github.com/apache/carbondata/pull/982 https://github.com/apache/carbondata/pull/986 also the same issue in different way. Please check. --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] carbondata pull request #986: [CARBONDATA-1108] Supported IUD for vector rea...

2017-05-31 Thread ravipesala
GitHub user ravipesala opened a pull request: https://github.com/apache/carbondata/pull/986 [CARBONDATA-1108] Supported IUD for vector reader in Spark 2.1 Current vector code does not support delete operation, this PR adds the delete functionality. You can merge this pull request

[GitHub] carbondata issue #982: [CARBONDATA-1112] fix failing IUD testcase

2017-05-31 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/982 Build Failed with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/2129/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] carbondata pull request #972: [WIP] Added set command in carbon to update pr...

2017-05-31 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/972#discussion_r119378513 --- Diff: integration/spark/src/main/scala/org/apache/spark/sql/hive/execution/command/CarbonHiveCommands.scala --- @@ -53,3 +55,15 @@ private[hive]

[GitHub] carbondata pull request #972: [WIP] Added set command in carbon to update pr...

2017-05-31 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/972#discussion_r119370617 --- Diff: core/src/main/java/org/apache/carbondata/core/util/SessionParams.java --- @@ -0,0 +1,148 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #972: [WIP] Added set command in carbon to update pr...

2017-05-31 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/972#discussion_r119370103 --- Diff: core/src/main/java/org/apache/carbondata/core/util/SessionParams.java --- @@ -0,0 +1,148 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #972: [WIP] Added set command in carbon to update pr...

2017-05-31 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/972#discussion_r119385679 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/CarbonHiveCommands.scala --- @@ -41,3 +43,36 @@ case class

[GitHub] carbondata pull request #972: [WIP] Added set command in carbon to update pr...

2017-05-31 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/972#discussion_r119369628 --- Diff: core/src/main/java/org/apache/carbondata/core/util/SessionParams.java --- @@ -0,0 +1,148 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #972: [WIP] Added set command in carbon to update pr...

2017-05-31 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/972#discussion_r119385878 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/CarbonHiveCommands.scala --- @@ -41,3 +43,36 @@ case class

[GitHub] carbondata pull request #972: [WIP] Added set command in carbon to update pr...

2017-05-31 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/972#discussion_r119383601 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/CarbonHiveCommands.scala --- @@ -41,3 +43,36 @@ case class

[GitHub] carbondata issue #985: [CARBONDATA-1090] added integration test cases for al...

2017-05-31 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/985 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

[GitHub] carbondata issue #979: [CARBONDATA-1102] resolved int,short type bug for hiv...

2017-05-31 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/979 add to whitelist --- 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

[GitHub] carbondata issue #979: [CARBONDATA-1102] resolved int,short type bug for hiv...

2017-05-31 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/979 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

[GitHub] carbondata issue #985: [CARBONDATA-1102] added integration test cases for al...

2017-05-31 Thread anubhav100
Github user anubhav100 commented on the issue: https://github.com/apache/carbondata/pull/985 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

[GitHub] carbondata issue #979: [CARBONDATA-1102] resolved int,short type bug for hiv...

2017-05-31 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/979 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

[GitHub] carbondata issue #962: Only for test ci(vacuous PR-don't merge)

2017-05-31 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/962 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

[GitHub] carbondata issue #981: [CARBONDATA-1111]Improve No dictionary column Include...

2017-05-31 Thread kumarvishal09
Github user kumarvishal09 commented on the issue: https://github.com/apache/carbondata/pull/981 add to whitelist --- 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

[GitHub] carbondata issue #982: [CARBONDATA-1112] fix failing IUD testcase

2017-05-31 Thread gvramana
Github user gvramana commented on the issue: https://github.com/apache/carbondata/pull/982 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

[GitHub] carbondata issue #978: [CARBONDATA-1109] Cover the case when last page is no...

2017-05-31 Thread kumarvishal09
Github user kumarvishal09 commented on the issue: https://github.com/apache/carbondata/pull/978 @watermen Can u please add a testcase --- 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

[GitHub] carbondata issue #985: [CARBONDATA-1102] added integration test cases for al...

2017-05-31 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/985 Build Failed with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/2128/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] carbondata issue #984: [CARBONDATA-1008] Make Caron table schema compatible ...

2017-05-31 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/984 Build Failed with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/2127/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] carbondata pull request #985: [CARBONDATA-1102] added integration test cases...

2017-05-31 Thread anubhav100
GitHub user anubhav100 opened a pull request: https://github.com/apache/carbondata/pull/985 [CARBONDATA-1102] added integration test cases for all data type in hive You can merge this pull request into a Git repository by running: $ git pull

[GitHub] carbondata issue #972: [WIP] Added set command in carbon to update propertie...

2017-05-31 Thread ManoharVanam
Github user ManoharVanam commented on the issue: https://github.com/apache/carbondata/pull/972 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

[GitHub] carbondata issue #984: [CARBONDATA-1008] Make Caron table schema compatible ...

2017-05-31 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/984 Build Failed with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/2126/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] carbondata issue #984: [CARBONDATA-1008] Make Caron table schema compatible ...

2017-05-31 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/984 ok to test --- 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,

[GitHub] carbondata issue #982: [CARBONDATA-1112] fix failing IUD testcase

2017-05-31 Thread ravipesala
Github user ravipesala commented on the issue: https://github.com/apache/carbondata/pull/982 add to whitelist --- 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

[jira] [Comment Edited] (CARBONDATA-1114) Failed to run tests in windows env

2017-05-31 Thread xuchuanyin (JIRA)
[ https://issues.apache.org/jira/browse/CARBONDATA-1114?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16031069#comment-16031069 ] xuchuanyin edited comment on CARBONDATA-1114 at 5/31/17 12:20 PM: --

[GitHub] carbondata pull request #984: [CARBONDATA-1008] Make Caron table schema comp...

2017-05-31 Thread anubhav100
Github user anubhav100 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/984#discussion_r119342650 --- Diff: integration/hive/src/main/scala/org/apache/carbondata/hiveexample/HiveExample.scala --- @@ -54,15 +54,15 @@ object HiveExample {

[jira] [Updated] (CARBONDATA-1114) Failed to run tests in windows env

2017-05-31 Thread xuchuanyin (JIRA)
[ https://issues.apache.org/jira/browse/CARBONDATA-1114?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] xuchuanyin updated CARBONDATA-1114: --- Request participants: (was: ) Description: when run build command "mvn

[jira] [Commented] (CARBONDATA-1114) Failed to run tests in windows env

2017-05-31 Thread xuchuanyin (JIRA)
[ https://issues.apache.org/jira/browse/CARBONDATA-1114?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16031069#comment-16031069 ] xuchuanyin commented on CARBONDATA-1114: it seems the tests failed due to operating files in

[jira] [Created] (CARBONDATA-1114) Failed to run tests in windows env

2017-05-31 Thread xuchuanyin (JIRA)
xuchuanyin created CARBONDATA-1114: -- Summary: Failed to run tests in windows env Key: CARBONDATA-1114 URL: https://issues.apache.org/jira/browse/CARBONDATA-1114 Project: CarbonData Issue

[GitHub] carbondata issue #962: Only for test ci(vacuous PR-don't merge)

2017-05-31 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/962 Build Failed with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/2125/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] carbondata issue #962: Only for test ci(vacuous PR-don't merge)

2017-05-31 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/962 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

[GitHub] carbondata issue #981: [CARBONDATA-1111]Improve No dictionary column Include...

2017-05-31 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/981 Build Failed with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/2123/ --- If your project is set up for it, you can reply to this email and have your

[GitHub] carbondata pull request #984: [CARBONDATA-1008] Make Caron table schema comp...

2017-05-31 Thread cenyuhai
GitHub user cenyuhai opened a pull request: https://github.com/apache/carbondata/pull/984 [CARBONDATA-1008] Make Caron table schema compatible with HIVE Make Caron table schema compatible with HIVE You can merge this pull request into a Git repository by running: $ git pull

[GitHub] carbondata pull request #983: [CARBONDATA-1113] Added validation for partiti...

2017-05-31 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/983#discussion_r119335023 --- Diff: integration/spark-common/src/main/scala/org/apache/carbondata/spark/util/CommonUtil.scala --- @@ -169,7 +169,7 @@ object CommonUtil {

[GitHub] carbondata issue #977: [CARBONDATA-1097] describe formatted query supported ...

2017-05-31 Thread rahulforallp
Github user rahulforallp commented on the issue: https://github.com/apache/carbondata/pull/977 @kumarvishal09 my query was was : > spark.sql( s""" | CREATE TABLE carbon_table( | shortField SHORT, | intField INT,

[GitHub] carbondata pull request #983: [CARBONDATA-1113] Added validation for partiti...

2017-05-31 Thread manishgupta88
GitHub user manishgupta88 opened a pull request: https://github.com/apache/carbondata/pull/983 [CARBONDATA-1113] Added validation for partition columns Add various validation to partition columns like:- 1. Partition columns cannot be drop using alter command 2. Range

[GitHub] carbondata pull request #982: [CARBONDATA-1112] fix failing IUD testcase

2017-05-31 Thread rahulforallp
GitHub user rahulforallp opened a pull request: https://github.com/apache/carbondata/pull/982 [CARBONDATA-1112] fix failing IUD testcase You can merge this pull request into a Git repository by running: $ git pull https://github.com/rahulforallp/incubator-carbondata

[jira] [Created] (CARBONDATA-1112) fix failing IUD test case

2017-05-31 Thread Rahul Kumar (JIRA)
Rahul Kumar created CARBONDATA-1112: --- Summary: fix failing IUD test case Key: CARBONDATA-1112 URL: https://issues.apache.org/jira/browse/CARBONDATA-1112 Project: CarbonData Issue Type: Bug

[GitHub] carbondata issue #962: Only for test ci(vacuous PR-don't merge)

2017-05-31 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/962 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

[GitHub] carbondata issue #981: [CARBONDATA-1111]Improve No dictionary column Include...

2017-05-31 Thread chenliang613
Github user chenliang613 commented on the issue: https://github.com/apache/carbondata/pull/981 add to whitelist --- 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

[GitHub] carbondata issue #981: [CARBONDATA-1111]Improve No dictionary column Include...

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

[GitHub] carbondata pull request #981: [CARBONDATA-1111]Improve No dictionary column ...

2017-05-31 Thread kumarvishal09
GitHub user kumarvishal09 opened a pull request: https://github.com/apache/carbondata/pull/981 [CARBONDATA-]Improve No dictionary column Include And Exclude filter Added binary search for No dictionary column in Include and Exclude filters You can merge this pull request into a

[jira] [Created] (CARBONDATA-1111) Improve No dictionary column Include And Exclude filter

2017-05-31 Thread kumar vishal (JIRA)
kumar vishal created CARBONDATA-: Summary: Improve No dictionary column Include And Exclude filter Key: CARBONDATA- URL: https://issues.apache.org/jira/browse/CARBONDATA- Project:

[GitHub] carbondata issue #979: resolved int,short type bug for hive ,modified the hi...

2017-05-31 Thread ravipesala
Github user ravipesala commented on the issue: https://github.com/apache/carbondata/pull/979 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

[GitHub] carbondata issue #973: [CARBONDATA-1107] Multi User load on same table is fa...

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

[GitHub] carbondata issue #977: [CARBONDATA-1097] describe formatted query supported ...

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

[GitHub] carbondata issue #964: [CARBONDATA-1099] Fixed bug for carbon-spark-shell in...

2017-05-31 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/964 Can one of the admins verify this patch? --- 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

[GitHub] carbondata issue #975: [Documentation] Single pass condition for high cardin...

2017-05-31 Thread CarbonDataQA
Github user CarbonDataQA commented on the issue: https://github.com/apache/carbondata/pull/975 Can one of the admins verify this patch? --- 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

[GitHub] carbondata pull request #969: [CARBONDATA-1077] ColumnDict and ALL_DICTIONAR...

2017-05-31 Thread kumarvishal09
Github user kumarvishal09 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/969#discussion_r119310777 --- Diff: integration/spark/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchema.scala --- @@ -438,8 +438,9 @@ case class

[GitHub] carbondata pull request #909: [CARBONDATA-1049] avoid logging raw data into ...

2017-05-31 Thread kumarvishal09
Github user kumarvishal09 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/909#discussion_r119307181 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonDataProcessorUtil.java --- @@ -522,4 +522,16 @@ public static

[GitHub] carbondata issue #977: [CARBONDATA-1097] describe formatted query supported ...

2017-05-31 Thread kumarvishal09
Github user kumarvishal09 commented on the issue: https://github.com/apache/carbondata/pull/977 @rahulforallp Can u please add one sample output in PR description --- 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

[GitHub] carbondata issue #966: [CARBONDATA-1101] Avoid widening between wrapper clas...

2017-05-31 Thread kumarvishal09
Github user kumarvishal09 commented on the issue: https://github.com/apache/carbondata/pull/966 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

[GitHub] carbondata pull request #952: [CARBONDATA-1094] Wrong results returned by th...

2017-05-31 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/952 --- 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

[GitHub] carbondata pull request #977: [CARBONDATA-1097] describe formatted query sup...

2017-05-31 Thread rahulforallp
GitHub user rahulforallp opened a pull request: https://github.com/apache/carbondata/pull/977 [CARBONDATA-1097] describe formatted query supported for no_inverted_Index You can merge this pull request into a Git repository by running: $ git pull

[jira] [Updated] (CARBONDATA-1109) Page lost in load process when last page is not be consumed at the end

2017-05-31 Thread Yadong Qi (JIRA)
[ https://issues.apache.org/jira/browse/CARBONDATA-1109?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Yadong Qi updated CARBONDATA-1109: -- Component/s: data-load > Page lost in load process when last page is not be consumed at

[jira] [Updated] (CARBONDATA-1109) Page lost in load process when last page is not be consumed at the end

2017-05-31 Thread Yadong Qi (JIRA)
[ https://issues.apache.org/jira/browse/CARBONDATA-1109?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Yadong Qi updated CARBONDATA-1109: -- Description: First, we use Producer-Consumer model in the write step, we have n(default

[jira] [Updated] (CARBONDATA-1109) Page lost in load process when last page is not be consumed at the end

2017-05-31 Thread Yadong Qi (JIRA)
[ https://issues.apache.org/jira/browse/CARBONDATA-1109?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Yadong Qi updated CARBONDATA-1109: -- Description: First, we use Producer-Consumer model in the write step, we have n(default

[jira] [Updated] (CARBONDATA-1109) Page lost in load process when last page is not be consumed at the end

2017-05-31 Thread Yadong Qi (JIRA)
[ https://issues.apache.org/jira/browse/CARBONDATA-1109?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Yadong Qi updated CARBONDATA-1109: -- Affects Version/s: 1.2.0 > Page lost in load process when last page is not be consumed at

[jira] [Created] (CARBONDATA-1109) Page lost in load process when last page is not be consumed at the end

2017-05-31 Thread Yadong Qi (JIRA)
Yadong Qi created CARBONDATA-1109: - Summary: Page lost in load process when last page is not be consumed at the end Key: CARBONDATA-1109 URL: https://issues.apache.org/jira/browse/CARBONDATA-1109

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119293599 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/row/DMAttribute.java --- @@ -0,0 +1,30 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119291017 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/DataMap.java --- @@ -0,0 +1,105 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119290612 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/DataMap.java --- @@ -0,0 +1,105 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119289157 --- Diff: core/src/main/java/org/apache/carbondata/core/datastore/impl/array/DataMapStoreFactory.java --- @@ -0,0 +1,93 @@ +/* + * Licensed to

[GitHub] carbondata pull request #719: [WIP][CARBONDATA-844] Avoid to get useless spl...

2017-05-31 Thread watermen
Github user watermen closed the pull request at: https://github.com/apache/carbondata/pull/719 --- 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

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119288412 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/schema/IndexSchema.java --- @@ -0,0 +1,37 @@ +/* + * Licensed to the Apache

[GitHub] carbondata issue #973: [CARBONDATA-1107] Multi User load on same table is fa...

2017-05-31 Thread gvramana
Github user gvramana commented on the issue: https://github.com/apache/carbondata/pull/973 LGTM --- 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

[GitHub] carbondata issue #940: [CARBONDATA-1081] Use binarySearch to replace for cla...

2017-05-31 Thread gvramana
Github user gvramana commented on the issue: https://github.com/apache/carbondata/pull/940 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

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119282300 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/DataMap.java --- @@ -0,0 +1,105 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119282288 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/DataMap.java --- @@ -0,0 +1,105 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119282180 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/DataMap.java --- @@ -0,0 +1,105 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #958: [CARBONDATA-1088] Added interfaces for Data Ma...

2017-05-31 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/958#discussion_r119279365 --- Diff: core/src/main/java/org/apache/carbondata/core/datastore/block/TableBlockInfo.java --- @@ -307,4 +316,45 @@ public void