[GitHub] carbondata pull request #1336: [CARBONDATA-1425] Inappropriate Exception dis...

2017-09-16 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1336#discussion_r139285031 --- Diff: integration/spark2/src/test/scala/org/apache/carbondata/spark/testsuite/partition/TestAlterPartitionTable.scala --- @@ -466,6

[GitHub] carbondata pull request #1336: [CARBONDATA-1425] Inappropriate Exception dis...

2017-09-16 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1336#discussion_r139284844 --- Diff: integration/spark2/src/test/scala/org/apache/carbondata/spark/testsuite/partition/TestAlterPartitionTable.scala --- @@ -36,7 +36,8

[GitHub] carbondata issue #1336: [CARBONDATA-1425] Inappropriate Exception displays w...

2017-09-15 Thread mayunSaicmotor
Github user mayunSaicmotor commented on the issue: https://github.com/apache/carbondata/pull/1336 retest this please ---

[GitHub] carbondata pull request #1336: [CARBONDATA-1425] Inappropriate Exception dis...

2017-09-06 Thread mayunSaicmotor
GitHub user mayunSaicmotor opened a pull request: https://github.com/apache/carbondata/pull/1336 [CARBONDATA-1425] Inappropriate Exception displays while creating a new partition with incorrect partition type change the error content when the range info data mismatch the partition

[GitHub] carbondata pull request #1193: [CARBONDATA-1327] Add carbon sort column exam...

2017-07-26 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1193#discussion_r129566524 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/CarbonSortColumnsExample.scala --- @@ -0,0 +1,126

[GitHub] carbondata pull request #1193: [CARBONDATA-1327] Add carbon sort column exam...

2017-07-24 Thread mayunSaicmotor
GitHub user mayunSaicmotor opened a pull request: https://github.com/apache/carbondata/pull/1193 [CARBONDATA-1327] Add carbon sort column examples Add carbon sort column examples and update the introduction document You can merge this pull request into a Git repository by running

[GitHub] carbondata pull request #1186: [WIP]Merge test code for AddColumnTestCases, ...

2017-07-19 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1186#discussion_r128272093 --- Diff: integration/spark2/src/test/scala/org/apache/spark/carbondata/restructure/vectorreader/AddColumnTestCases.scala --- @@ -42,26 +43,55

[GitHub] carbondata pull request #1186: [WIP]Merge test code for AddColumnTestCases, ...

2017-07-19 Thread mayunSaicmotor
GitHub user mayunSaicmotor opened a pull request: https://github.com/apache/carbondata/pull/1186 [WIP]Merge test code for AddColumnTestCases, DropColumnTestCases and ChangeDataTypeTestCases merge test code for AddColumnTestCases, DropColumnTestCases and ChangeDataTypeTestCases

[GitHub] carbondata pull request #1159: [CARBONDATA-1274] Adds example for update and...

2017-07-12 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1159#discussion_r127007813 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/DataUpdateDeleteExample.scala --- @@ -0,0 +1,183

[GitHub] carbondata pull request #988: [CARBONDATA-1110] put if clause out of the for...

2017-07-12 Thread mayunSaicmotor
Github user mayunSaicmotor closed the pull request at: https://github.com/apache/carbondata/pull/988 --- 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 pull request #1159: [CARBONDATA-1274] added example for update an...

2017-07-11 Thread mayunSaicmotor
GitHub user mayunSaicmotor opened a pull request: https://github.com/apache/carbondata/pull/1159 [CARBONDATA-1274] added example for update and delete added detailed examples for update and delete in spark 2.1 and spark1.6 You can merge this pull request into a Git repository

[GitHub] carbondata pull request #1103: [WIP] Implement range interval partition

2017-07-10 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1103#discussion_r126377929 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/CarbonPartitionExample.scala --- @@ -113,14 +113,31 @@ object

[GitHub] carbondata pull request #1103: [WIP] Implement range interval partition

2017-07-10 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1103#discussion_r126377354 --- Diff: integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/partition/TestDDLForPartitionTable.scala

[GitHub] carbondata pull request #1103: [WIP] Implement range interval partition

2017-07-10 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1103#discussion_r126376641 --- Diff: integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/partition/TestDDLForPartitionTable.scala

[GitHub] carbondata pull request #1103: [WIP] Implement range interval partition

2017-07-03 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1103#discussion_r125289242 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/schema/PartitionInfo.java --- @@ -47,6 +47,12 @@ */ private int

[GitHub] carbondata pull request #1094: [CARBONDATA-1181] Show partitions

2017-06-28 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1094#discussion_r124582318 --- Diff: examples/spark/src/main/scala/org/apache/carbondata/examples/CarbonPartitionExample.scala --- @@ -0,0 +1,128

[GitHub] carbondata pull request #1094: [CARBONDATA-1181] Show partitions

2017-06-27 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1094#discussion_r124206197 --- Diff: examples/spark/src/main/scala/org/apache/carbondata/examples/CarbonPartitionExample.scala --- @@ -0,0 +1,128

[GitHub] carbondata pull request #1094: [CARBONDATA-1181] Show partitions

2017-06-27 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1094#discussion_r124204888 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/DDLStrategy.scala --- @@ -117,6 +117,8 @@ class DDLStrategy

[GitHub] carbondata pull request #1094: [CARBONDATA-1181] Show partitions

2017-06-27 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1094#discussion_r124204837 --- Diff: examples/spark/src/main/scala/org/apache/carbondata/examples/CarbonPartitionExample.scala --- @@ -0,0 +1,128

[GitHub] carbondata pull request #1094: [CARBONDATA-1181] Show partitions

2017-06-24 Thread mayunSaicmotor
GitHub user mayunSaicmotor opened a pull request: https://github.com/apache/carbondata/pull/1094 [CARBONDATA-1181] Show partitions Show table partition information You can merge this pull request into a Git repository by running: $ git pull https://github.com/mayunSaicmotor

[GitHub] carbondata pull request #1093: [CARBONDATA-1181] Show partitions

2017-06-24 Thread mayunSaicmotor
Github user mayunSaicmotor closed the pull request at: https://github.com/apache/carbondata/pull/1093 --- 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 pull request #1093: [CARBONDATA-1181] Show partitions

2017-06-24 Thread mayunSaicmotor
GitHub user mayunSaicmotor reopened a pull request: https://github.com/apache/carbondata/pull/1093 [CARBONDATA-1181] Show partitions Show table partition information You can merge this pull request into a Git repository by running: $ git pull https://github.com/mayunSaicmotor

[GitHub] carbondata pull request #1093: [CARBONDATA-1181] Show partitions

2017-06-24 Thread mayunSaicmotor
Github user mayunSaicmotor closed the pull request at: https://github.com/apache/carbondata/pull/1093 --- 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 pull request #1042: [CARBONDATA-1181] Show partitions

2017-06-24 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1042#discussion_r123875651 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/ShowPartitionInfoExample.scala --- @@ -0,0 +1,110

[GitHub] carbondata pull request #1042: [CARBONDATA-1181] Show partitions

2017-06-23 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1042#discussion_r123868164 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonAnalysisRules.scala --- @@ -203,6 +205,7 @@ case class

[GitHub] carbondata pull request #1042: [CARBONDATA-1181] Show partitions

2017-06-23 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1042#discussion_r123867997 --- Diff: integration/spark/src/main/scala/org/apache/spark/sql/hive/CarbonAnalysisRules.scala --- @@ -170,6 +171,7 @@ object

[GitHub] carbondata pull request #988: [CARBONDATA-1110] put if clause out of the for...

2017-06-11 Thread mayunSaicmotor
GitHub user mayunSaicmotor reopened a pull request: https://github.com/apache/carbondata/pull/988 [CARBONDATA-1110] put if clause out of the for clause it should be better to put if clause out of the for clause You can merge this pull request into a Git repository by running

[GitHub] carbondata pull request #1010: [CARBONDATA-1110] put if clause out of the fo...

2017-06-08 Thread mayunSaicmotor
GitHub user mayunSaicmotor opened a pull request: https://github.com/apache/carbondata/pull/1010 [CARBONDATA-1110] put if clause out of the for clause it should be better to put if clause out of the for clause You can merge this pull request into a Git repository by running

[GitHub] carbondata issue #980: [CARBONDATA-1110] put if clause out of the for clause

2017-06-01 Thread mayunSaicmotor
Github user mayunSaicmotor commented on the issue: https://github.com/apache/carbondata/pull/980 closed this pr, raised a new pr https://github.com/apache/carbondata/pull/988 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] carbondata pull request #988: [CARBONDATA-1110] put if clause out of the for...

2017-06-01 Thread mayunSaicmotor
GitHub user mayunSaicmotor opened a pull request: https://github.com/apache/carbondata/pull/988 [CARBONDATA-1110] put if clause out of the for clause it should be better to put if clause out of the for clause You can merge this pull request into a Git repository by running

[GitHub] carbondata pull request #980: [CARBONDATA-1110] put if clause out of the for...

2017-06-01 Thread mayunSaicmotor
Github user mayunSaicmotor closed the pull request at: https://github.com/apache/carbondata/pull/980 --- 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 pull request #940: [CARBONDATA-1081] Use binarySearch to replace ...

2017-05-29 Thread mayunSaicmotor
GitHub user mayunSaicmotor reopened a pull request: https://github.com/apache/carbondata/pull/940 [CARBONDATA-1081] Use binarySearch to replace for clause to improve performance use binarySearch to replace for clause in method setFilterdIndexToBitSet

[GitHub] carbondata pull request #968: [CARBONDATA-1081] Use binarySearch to replace ...

2017-05-29 Thread mayunSaicmotor
Github user mayunSaicmotor closed the pull request at: https://github.com/apache/carbondata/pull/968 --- 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 pull request #968: [CARBONDATA-1081] Use binarySearch to replace ...

2017-05-29 Thread mayunSaicmotor
GitHub user mayunSaicmotor opened a pull request: https://github.com/apache/carbondata/pull/968 [CARBONDATA-1081] Use binarySearch to replace for clause to improve performance Use binarySearch to replace for clause to improve performance in ExcludeFilterExecuterImpl.java You can

[GitHub] carbondata pull request #940: [CARBONDATA-1081] Use binarySearch to replace ...

2017-05-29 Thread mayunSaicmotor
Github user mayunSaicmotor closed the pull request at: https://github.com/apache/carbondata/pull/940 --- 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 pull request #940: [CARBONDATA-1081] Use binarySearch to replace ...

2017-05-29 Thread mayunSaicmotor
GitHub user mayunSaicmotor reopened a pull request: https://github.com/apache/carbondata/pull/940 [CARBONDATA-1081] Use binarySearch to replace for clause to improve performance use binarySearch to replace for clause in method setFilterdIndexToBitSet

[GitHub] carbondata pull request #940: [CARBONDATA-1081] Use binarySearch to replace ...

2017-05-29 Thread mayunSaicmotor
Github user mayunSaicmotor closed the pull request at: https://github.com/apache/carbondata/pull/940 --- 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 pull request #940: [CARBONDATA-1081]use binarySearch to replace...

2017-05-24 Thread mayunSaicmotor
GitHub user mayunSaicmotor opened a pull request: https://github.com/apache/carbondata/pull/940 ï¼»CARBONDATA-1081]use binarySearch to replace for clause to improve performance use binarySearch to replace for clause in method setFilterdIndexToBitSet

[GitHub] carbondata pull request #938: [CARBONDATA-1079]change positon to position

2017-05-23 Thread mayunSaicmotor
GitHub user mayunSaicmotor opened a pull request: https://github.com/apache/carbondata/pull/938 [CARBONDATA-1079]change positon to position [CARBONDATA-1079]change positon to position You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] carbondata pull request #877: [CARBONDATA-951] create table ddl can specify ...

2017-05-11 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/carbondata/pull/877#discussion_r115936997 --- Diff: integration/spark/src/test/scala/org/apache/carbondata/spark/testsuite/createtable/TestCreateTableSyntax.scala --- @@ -31,7 +36,60

[GitHub] incubator-carbondata pull request #833: [CARBONDATA-951] create table ddl ca...

2017-04-30 Thread mayunSaicmotor
Github user mayunSaicmotor commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/833#discussion_r114095485 --- Diff: format/src/main/thrift/carbondata.thrift --- @@ -130,6 +130,7 @@ struct DataChunk2{ 9: optional list encoder_meta

[GitHub] incubator-carbondata pull request #848: [CARBONDATA-984] change columnSchenm...

2017-04-25 Thread mayunSaicmotor
GitHub user mayunSaicmotor opened a pull request: https://github.com/apache/incubator-carbondata/pull/848 [CARBONDATA-984] change columnSchenma to columnSchema change columnSchenma to columnSchema You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-carbondata pull request #840: [CARBONDATA-975] remove unreasonable...

2017-04-24 Thread mayunSaicmotor
GitHub user mayunSaicmotor opened a pull request: https://github.com/apache/incubator-carbondata/pull/840 [CARBONDATA-975] remove unreasonable code remove the first "encoders.remove(" from the below code in carbonTableSchema.scala encoders.remove(encod

[GitHub] incubator-carbondata issue #833: [CARBONDATA-951] create table ddl can speci...

2017-04-23 Thread mayunSaicmotor
Github user mayunSaicmotor commented on the issue: https://github.com/apache/incubator-carbondata/pull/833 fixed for the below items 1.the column should use DICTIONARY encoding also. 2.the column will be a dimension, maybe sorted column or unsorted column --- If your project

[GitHub] incubator-carbondata pull request #833: [CARBONDATA-951] create table ddl ca...

2017-04-21 Thread mayunSaicmotor
GitHub user mayunSaicmotor opened a pull request: https://github.com/apache/incubator-carbondata/pull/833 [CARBONDATA-951] create table ddl can specify a bitmap option and add a new Encoding: BITMAP create table ddl can specify a bitmap option add a new Encoding: BITMAP
â

[GitHub] incubator-carbondata pull request #832: [CARBONDATA-951] create table ddl ca...

2017-04-21 Thread mayunSaicmotor
Github user mayunSaicmotor closed the pull request at: https://github.com/apache/incubator-carbondata/pull/832 --- 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] incubator-carbondata pull request #827: [CARBONDATA-951] create table ddl ca...

2017-04-20 Thread mayunSaicmotor
GitHub user mayunSaicmotor opened a pull request: https://github.com/apache/incubator-carbondata/pull/827 [CARBONDATA-951] create table ddl can specify a bitmap option and add a new Encoding: BITMAP

 1. create table ddl can specify a bitmap option 2. add a new Encoding