[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-07 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131635037 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonDataProcessorUtil.java --- @@ -181,9 +181,11 @@ public static void

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-07 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131634496 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchema.scala --- @@ -183,6 +190,138 @@ case class

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-07 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131634414 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchema.scala --- @@ -183,6 +190,138 @@ case class

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-07 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131630948 --- Diff: integration/spark-common/src/main/scala/org/apache/carbondata/spark/rdd/CarbonScanPartitionRDD.scala --- @@ -0,0 +1,267 @@ +/* + *

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-03 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131067233 --- Diff: processing/src/main/java/org/apache/carbondata/processing/spliter/CarbonDataSpliterUtil.java --- @@ -0,0 +1,40 @@ +/* + * Licensed

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-03 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131067082 --- Diff: hadoop/src/main/java/org/apache/carbondata/hadoop/api/CarbonTableInputFormat.java --- @@ -321,6 +321,84 @@ private AbsoluteTableIdentifier

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-03 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131067032 --- Diff: hadoop/src/main/java/org/apache/carbondata/hadoop/api/CarbonTableInputFormat.java --- @@ -321,6 +321,84 @@ private AbsoluteTableIdentifier

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-03 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131063903 --- Diff: hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java --- @@ -107,6 +107,7 @@ // comma separated list of input

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-03 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131063552 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/CarbonPartitionExample.scala --- @@ -101,17 +126,40 @@ object

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-03 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131063435 --- Diff: conf/carbon.properties.template --- @@ -42,6 +42,9 @@ carbon.enableXXHash=true #carbon.max.level.cache.size=-1 #enable prefetch of

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-03 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131059761 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchema.scala --- @@ -184,6 +189,161 @@ case class

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-03 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131059448 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchema.scala --- @@ -184,6 +189,161 @@ case class

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-03 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131059447 --- Diff: integration/spark2/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataRDDFactory.scala --- @@ -345,6 +365,65 @@ object

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-03 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131059444 --- Diff: integration/spark2/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataRDDFactory.scala --- @@ -345,6 +365,65 @@ object

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-02 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131058343 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchema.scala --- @@ -184,6 +189,161 @@ case class

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-02 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131058295 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchema.scala --- @@ -184,6 +189,161 @@ case class

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-02 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131058061 --- Diff: processing/src/main/java/org/apache/carbondata/processing/spliter/CarbonDataSpliterUtil.java --- @@ -0,0 +1,40 @@ +/* + * Licensed

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-02 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131057576 --- Diff: hadoop/src/main/java/org/apache/carbondata/hadoop/api/CarbonTableInputFormat.java --- @@ -321,6 +321,84 @@ private AbsoluteTableIdentifier

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-02 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131057207 --- Diff: hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java --- @@ -107,6 +107,7 @@ // comma separated list of input

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-02 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131057020 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/CarbonPartitionExample.scala --- @@ -101,17 +126,40 @@ object

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-08-02 Thread jackylk
Github user jackylk commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r131056387 --- Diff: conf/carbon.properties.template --- @@ -42,6 +42,9 @@ carbon.enableXXHash=true #carbon.max.level.cache.size=-1 #enable prefetch of

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-31 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r130506636 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/schema/PartitionInfo.java --- @@ -65,6 +65,31 @@ public PartitionInfo(List

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-31 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r130506012 --- Diff: core/src/main/java/org/apache/carbondata/core/util/path/CarbonTablePath.java --- @@ -303,6 +303,20 @@ public String

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-31 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r130505076 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/schema/PartitionInfo.java --- @@ -65,6 +65,31 @@ public PartitionInfo(List

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-31 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r130504413 --- Diff: hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java --- @@ -440,9 +510,17 @@ protected Expression

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-31 Thread xuchuanyin
Github user xuchuanyin commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r130501996 --- Diff: core/src/main/java/org/apache/carbondata/core/util/path/CarbonTablePath.java --- @@ -303,6 +303,20 @@ public String

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-31 Thread xuchuanyin
Github user xuchuanyin commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r130501212 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/schema/PartitionInfo.java --- @@ -65,6 +65,31 @@ public PartitionInfo(List

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-31 Thread xuchuanyin
Github user xuchuanyin commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r130503124 --- Diff: hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java --- @@ -440,9 +510,17 @@ protected Expression

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129744678 --- Diff: core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java --- @@ -308,6 +308,10 @@ @CarbonProperty

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129740890 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/CarbonPartitionExample.scala --- @@ -101,17 +126,40 @@ object

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread chenerlu
Github user chenerlu commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129598686 --- Diff: hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java --- @@ -440,9 +510,16 @@ protected Expression

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread chenerlu
Github user chenerlu commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129598531 --- Diff: hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java --- @@ -440,9 +510,16 @@ protected Expression

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread chenerlu
Github user chenerlu commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129584623 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/CarbonPartitionExample.scala --- @@ -101,17 +126,40 @@ object

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread chenerlu
Github user chenerlu commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129583765 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/result/iterator/PartitionSpliterRawResultIterator.java --- @@ -0,0 +1,95 @@ +/*

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread chenerlu
Github user chenerlu commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129583246 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/result/iterator/PartitionSpliterRawResultIterator.java --- @@ -0,0 +1,95 @@ +/*

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129583064 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/schema/PartitionInfo.java --- @@ -65,6 +65,31 @@ public PartitionInfo(List

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread chenerlu
Github user chenerlu commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129582365 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/result/iterator/PartitionSpliterRawResultIterator.java --- @@ -0,0 +1,95 @@ +/*

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread chenerlu
Github user chenerlu commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129582258 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/result/iterator/PartitionSpliterRawResultIterator.java --- @@ -0,0 +1,95 @@ +/*

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread chenerlu
Github user chenerlu commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129582111 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/result/iterator/PartitionSpliterRawResultIterator.java --- @@ -0,0 +1,95 @@ +/*

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread chenerlu
Github user chenerlu commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129534273 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/schema/PartitionInfo.java --- @@ -65,6 +65,31 @@ public PartitionInfo(List

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread chenerlu
Github user chenerlu commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129533186 --- Diff: core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java --- @@ -308,6 +308,10 @@ @CarbonProperty

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129516143 --- Diff: integration/spark-common-test/src/test/resources/partition_data.csv --- @@ -0,0 +1,27 @@

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129515426 --- Diff: conf/carbon.properties.template --- @@ -42,6 +42,9 @@ carbon.enableXXHash=true #carbon.max.level.cache.size=-1 #enable prefetch of

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129514953 --- Diff: integration/spark-common-test/src/test/resources/partition_data.csv --- @@ -0,0 +1,27 @@

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread chenliang613
Github user chenliang613 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129514137 --- Diff: conf/carbon.properties.template --- @@ -42,6 +42,9 @@ carbon.enableXXHash=true #carbon.max.level.cache.size=-1 #enable prefetch

[GitHub] carbondata pull request #1192: [CARBONDATA-940] alter table add/split partit...

2017-07-26 Thread chenliang613
Github user chenliang613 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1192#discussion_r129513477 --- Diff: integration/spark-common-test/src/test/resources/partition_data.csv --- @@ -0,0 +1,27 @@