[GitHub] carbondata pull request #3070: [CARBONDATA-3246]Fix sdk reader issue if batc...

2019-01-11 Thread qiuchenjian
Github user qiuchenjian commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3070#discussion_r247292846 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/result/iterator/AbstractDetailQueryResultIterator.java --- @@ -94,6 +94,9 @@

[GitHub] carbondata pull request #3066: [CARBONDATA-3244] Add benchmark for Change Da...

2019-01-11 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3066#discussion_r247119761 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/benchmark/CDCBenchmark.scala --- @@ -0,0 +1,256 @@ +/* + * Licensed to the

[GitHub] carbondata pull request #3064: [CARBONDATA-3243] Updated DOC for No-Sort Com...

2019-01-11 Thread NamanRastogi
Github user NamanRastogi commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3064#discussion_r247097474 --- Diff: integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala --- @@ -1201,6 +1202,17 @@ abstract

[GitHub] carbondata pull request #3064: [CARBONDATA-3243] Updated DOC for No-Sort Com...

2019-01-11 Thread shardul-cr7
Github user shardul-cr7 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3064#discussion_r247096965 --- Diff: integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala --- @@ -1201,6 +1202,17 @@ abstract class

[GitHub] carbondata pull request #3070: [CARBONDATA-3246]Fix sdk reader issue if batc...

2019-01-11 Thread shardul-cr7
GitHub user shardul-cr7 opened a pull request: https://github.com/apache/carbondata/pull/3070 [CARBONDATA-3246]Fix sdk reader issue if batch size is given as zero and vectorRead False. This PR is to fix sdk reader issue when batch size is given as zero and vectorRead False.

[GitHub] carbondata pull request #3069: [WIP][CARBONDATA-3232] Add test case for allu...

2019-01-11 Thread xubo245
GitHub user xubo245 opened a pull request: https://github.com/apache/carbondata/pull/3069 [WIP][CARBONDATA-3232] Add test case for alluxio UT Add test case for alluxio UT 1. install alluxio UT environment, start alluxio minicluster before running test case. 2. add

[GitHub] carbondata pull request #3068: [HOTFIX] Fixed NPE during query with Local Di...

2019-01-11 Thread kumarvishal09
GitHub user kumarvishal09 opened a pull request: https://github.com/apache/carbondata/pull/3068 [HOTFIX] Fixed NPE during query with Local Dictionary **Problem:** Query is failing with NPE when some blocklet encoded with local dictionary and some without local dictionary.

[GitHub] carbondata pull request #3064: [CARBONDATA-3243] Updated DOC for No-Sort Com...

2019-01-10 Thread NamanRastogi
Github user NamanRastogi commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3064#discussion_r247011216 --- Diff: docs/dml-of-carbondata.md --- @@ -106,6 +107,13 @@ CarbonData DML statements are documented here,which includes:

[GitHub] carbondata pull request #3064: [CARBONDATA-3243] Updated DOC for No-Sort Com...

2019-01-10 Thread NamanRastogi
Github user NamanRastogi commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3064#discussion_r247010700 --- Diff: docs/dml-of-carbondata.md --- @@ -49,6 +49,7 @@ CarbonData DML statements are documented here,which includes: |

[GitHub] carbondata pull request #3064: [CARBONDATA-3243] Updated DOC for No-Sort Com...

2019-01-10 Thread NamanRastogi
Github user NamanRastogi commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3064#discussion_r247010601 --- Diff: docs/configuration-parameters.md --- @@ -208,6 +208,7 @@ RESET | carbon.options.date.format | Specifies the data

[GitHub] carbondata pull request #3033: [CARBONDATA-3215] Optimize the documentation

2019-01-10 Thread sraghunandan
Github user sraghunandan commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3033#discussion_r247000712 --- Diff: docs/datamap-developer-guide.md --- @@ -19,16 +19,16 @@ ### Introduction DataMap is a data structure that can be used to

[GitHub] carbondata pull request #3065: [HOTFIX] Optimize presto-guide

2019-01-10 Thread qiuchenjian
Github user qiuchenjian commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3065#discussion_r246997808 --- Diff: docs/presto-guide.md --- @@ -220,7 +220,8 @@ Now you can use the Presto CLI on the coordinator to query data sources in the c

[GitHub] carbondata pull request #3065: [HOTFIX] Optimize presto-guide

2019-01-10 Thread chenliang613
Github user chenliang613 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3065#discussion_r246996198 --- Diff: docs/presto-guide.md --- @@ -220,7 +220,8 @@ Now you can use the Presto CLI on the coordinator to query data sources in the c

[GitHub] carbondata pull request #3066: [CARBONDATA-3244] Add benchmark for Change Da...

2019-01-10 Thread qiuchenjian
Github user qiuchenjian commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3066#discussion_r246995496 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/benchmark/CDCBenchmark.scala --- @@ -0,0 +1,256 @@ +/* + * Licensed to the

[GitHub] carbondata pull request #3067: [HOTFIX] Fix compile error after merging PR#3...

2019-01-10 Thread zzcclp
GitHub user zzcclp opened a pull request: https://github.com/apache/carbondata/pull/3067 [HOTFIX] Fix compile error after merging PR#3001 Fix compile error after merging PR#3001 Be sure to do all of the following checklist to help us incorporate your contribution

[GitHub] carbondata pull request #3063: [CARBONDATA-3242] Move Range_Column into the ...

2019-01-10 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3063#discussion_r246979073 --- Diff: core/src/main/java/org/apache/carbondata/core/util/CarbonProperties.java --- @@ -1364,6 +1365,29 @@ public int

[GitHub] carbondata pull request #3063: [CARBONDATA-3242] Move Range_Column into the ...

2019-01-10 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3063#discussion_r246978328 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/schema/table/CarbonTable.java --- @@ -947,6 +947,16 @@ public int

[GitHub] carbondata pull request #3066: [CARBONDATA-3244] Add benchmark for Change Da...

2019-01-10 Thread qiuchenjian
Github user qiuchenjian commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3066#discussion_r246977629 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/benchmark/CDCBenchmark.scala --- @@ -0,0 +1,256 @@ +/* + * Licensed to the

[GitHub] carbondata pull request #3064: [CARBONDATA-3243] Updated DOC for No-Sort Com...

2019-01-10 Thread qiuchenjian
Github user qiuchenjian commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3064#discussion_r246975368 --- Diff: docs/dml-of-carbondata.md --- @@ -106,6 +107,13 @@ CarbonData DML statements are documented here,which includes:

[GitHub] carbondata pull request #3064: [CARBONDATA-3243] Updated DOC for No-Sort Com...

2019-01-10 Thread qiuchenjian
Github user qiuchenjian commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3064#discussion_r246975324 --- Diff: docs/dml-of-carbondata.md --- @@ -49,6 +49,7 @@ CarbonData DML statements are documented here,which includes: |

[GitHub] carbondata pull request #3064: [CARBONDATA-3243] Updated DOC for No-Sort Com...

2019-01-10 Thread qiuchenjian
Github user qiuchenjian commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3064#discussion_r246975024 --- Diff: docs/configuration-parameters.md --- @@ -208,6 +208,7 @@ RESET | carbon.options.date.format | Specifies the data

[GitHub] carbondata pull request #3063: [CARBONDATA-3242] Move Range_Column into the ...

2019-01-10 Thread qiuchenjian
Github user qiuchenjian commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3063#discussion_r246973842 --- Diff: core/src/main/java/org/apache/carbondata/core/util/CarbonProperties.java --- @@ -1364,6 +1365,29 @@ public int

[GitHub] carbondata pull request #3063: [CARBONDATA-3242] Move Range_Column into the ...

2019-01-10 Thread qiuchenjian
Github user qiuchenjian commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3063#discussion_r246973727 --- Diff: core/src/main/java/org/apache/carbondata/core/util/CarbonProperties.java --- @@ -1364,6 +1365,29 @@ public int

[GitHub] carbondata pull request #3063: [CARBONDATA-3242] Move Range_Column into the ...

2019-01-10 Thread qiuchenjian
Github user qiuchenjian commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3063#discussion_r246973311 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/schema/table/CarbonTable.java --- @@ -947,6 +947,16 @@ public int

[GitHub] carbondata pull request #3065: Optimize presto-guide

2019-01-10 Thread qiuchenjian
Github user qiuchenjian commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3065#discussion_r246972785 --- Diff: docs/presto-guide.md --- @@ -220,7 +220,8 @@ Now you can use the Presto CLI on the coordinator to query data sources in the c

[GitHub] carbondata pull request #3066: [CARBONDATA-3244] Add benchmark for Change Da...

2019-01-10 Thread jackylk
GitHub user jackylk opened a pull request: https://github.com/apache/carbondata/pull/3066 [CARBONDATA-3244] Add benchmark for Change Data Capture scenario CDC (change data capture) is a common scenario for analyzing slowly changed table in data warehouse. It is good to add

[GitHub] carbondata pull request #3059: [HOTFIX][DataLoad]fix task assignment issue u...

2019-01-10 Thread KanakaKumar
Github user KanakaKumar commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3059#discussion_r246826911 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonLoaderUtil.java --- @@ -609,6 +609,14 @@ public static Dictionary

[GitHub] carbondata pull request #3065: Optimize presto-guide

2019-01-10 Thread chenliang613
GitHub user chenliang613 opened a pull request: https://github.com/apache/carbondata/pull/3065 Optimize presto-guide Some users made mistake: copy the assemble jar. Add more description to clarify, need to copy many jars from

[GitHub] carbondata pull request #3064: [CARBONDATA-3243] Updated DOC for No-Sort Com...

2019-01-10 Thread NamanRastogi
Github user NamanRastogi commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3064#discussion_r246780308 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/hive/execution/command/CarbonHiveCommands.scala --- @@ -127,6 +127,9 @@ object

[GitHub] carbondata pull request #3064: [WIP] Updated DOC for No-Sort Compaction and ...

2019-01-10 Thread kunal642
Github user kunal642 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3064#discussion_r246777849 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/hive/execution/command/CarbonHiveCommands.scala --- @@ -127,6 +127,9 @@ object

[GitHub] carbondata pull request #3064: [WIP] Updated DOC for No-Sort Compaction and ...

2019-01-10 Thread NamanRastogi
GitHub user NamanRastogi opened a pull request: https://github.com/apache/carbondata/pull/3064 [WIP] Updated DOC for No-Sort Compaction and a few Fixes 1. Updated Doc 2. Checking SORT_SCOPE in session property CARBON.TABLE.LOAD.SORT.SCOPE in CarbonTable.getSortScope() 3.

[GitHub] carbondata pull request #3063: [CARBONDATA-3242] Move Range_Column into the ...

2019-01-10 Thread QiangCai
GitHub user QiangCai opened a pull request: https://github.com/apache/carbondata/pull/3063 [CARBONDATA-3242] Move Range_Column into the table level properties Move Range_Column into the table level properties Be sure to do all of the following checklist to help us

[GitHub] carbondata pull request #3001: [CARBONDATA-3220] Support presto to read stre...

2019-01-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/3001 ---

[GitHub] carbondata pull request #3062: [WIP][CARBONDATA-3241] Refactor the requested...

2019-01-10 Thread qiuchenjian
Github user qiuchenjian commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3062#discussion_r246715623 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/strategy/CarbonLateDecodeStrategy.scala --- @@ -449,9 +455,10 @@

[GitHub] carbondata pull request #3062: [WIP][CARBONDATA-3241] Refactor the requested...

2019-01-10 Thread dhatchayani
GitHub user dhatchayani opened a pull request: https://github.com/apache/carbondata/pull/3062 [WIP][CARBONDATA-3241] Refactor the requested scan columns and the projection columns Refactor the requested columns methods by changing both the scan list and the projection list

[GitHub] carbondata pull request #3061: [CARBONDATA-3239] Fix ArrayIndexOutOfBoundsEx...

2019-01-10 Thread QiangCai
GitHub user QiangCai opened a pull request: https://github.com/apache/carbondata/pull/3061 [CARBONDATA-3239] Fix ArrayIndexOutOfBoundsException in DataSkewRangePartitioner Fix ArrayIndexOutOfBoundsException in DataSkewRangePartitioner and add test case Be sure to do all

[GitHub] carbondata pull request #3023: [CARBONDATA-3197][BloomDataMap] Include bloom...

2019-01-10 Thread kevinjmh
Github user kevinjmh commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3023#discussion_r246663278 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/CarbonEnv.scala --- @@ -184,6 +184,9 @@ object CarbonEnv {

[GitHub] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2019-01-09 Thread BJangir
Github user BJangir commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2991#discussion_r246647432 --- Diff: docs/csdk-guide.md --- @@ -29,6 +29,32 @@ code and without CarbonSession. In the carbon jars package, there exist a

[GitHub] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2019-01-09 Thread BJangir
Github user BJangir commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2991#discussion_r246645671 --- Diff: docs/csdk-guide.md --- @@ -40,6 +66,7 @@ release the memory and destroy JVM. C++ SDK support read batch row. User can set batch by

[GitHub] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2019-01-09 Thread BJangir
Github user BJangir commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2991#discussion_r246644993 --- Diff: docs/csdk-guide.md --- @@ -29,6 +29,32 @@ code and without CarbonSession. In the carbon jars package, there exist a

[GitHub] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2019-01-09 Thread BJangir
Github user BJangir commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2991#discussion_r246644367 --- Diff: docs/csdk-guide.md --- @@ -29,6 +29,32 @@ code and without CarbonSession. In the carbon jars package, there exist a

[GitHub] carbondata pull request #2991: [CARBONDATA-3043] Add build script and add te...

2019-01-09 Thread BJangir
Github user BJangir commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2991#discussion_r246643403 --- Diff: docs/csdk-guide.md --- @@ -29,6 +29,32 @@ code and without CarbonSession. In the carbon jars package, there exist a

[GitHub] carbondata pull request #3054: [CARBONDATA-3232] Add example and doc for all...

2019-01-09 Thread xuchuanyin
Github user xuchuanyin commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3054#discussion_r246427391 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/AlluxioExample.scala --- @@ -28,46 +33,86 @@ import

[GitHub] carbondata pull request #3060: [HOTFIX] Exclude filter doesn't work in prest...

2019-01-09 Thread ajantha-bhat
GitHub user ajantha-bhat opened a pull request: https://github.com/apache/carbondata/pull/3060 [HOTFIX] Exclude filter doesn't work in presto carbon in cluster **problem:** exclude filter fails in cluster for presto carbon with exception. ``` java.lang.NoClassDefFoundError:

[GitHub] carbondata pull request #3059: [HOTFIX][DataLoad]fix task assignment issue u...

2019-01-09 Thread kevinjmh
Github user kevinjmh commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3059#discussion_r246376652 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonLoaderUtil.java --- @@ -609,6 +609,14 @@ public static Dictionary

[GitHub] carbondata pull request #3055: [CARBONDATA-3237] Fix presto carbon issues in...

2019-01-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/3055 ---

[GitHub] carbondata pull request #3029: [CARBONDATA-3200] No-Sort compaction

2019-01-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/3029 ---

[GitHub] carbondata pull request #3059: [HOTFIX][DataLoad]fix task assignment issue u...

2019-01-09 Thread qiuchenjian
Github user qiuchenjian commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3059#discussion_r246352937 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonLoaderUtil.java --- @@ -609,6 +609,14 @@ public static Dictionary

[GitHub] carbondata pull request #3056: [CARBONDATA-3236] Fix for JVM Crash for inser...

2019-01-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/3056 ---

[GitHub] carbondata pull request #3029: [CARBONDATA-3200] No-Sort compaction

2019-01-09 Thread NamanRastogi
Github user NamanRastogi commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3029#discussion_r246329602 --- Diff: processing/src/main/java/org/apache/carbondata/processing/merger/CarbonCompactionUtil.java --- @@ -400,24 +417,53 @@ private static int

[GitHub] carbondata pull request #3029: [CARBONDATA-3200] No-Sort compaction

2019-01-09 Thread NamanRastogi
Github user NamanRastogi commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3029#discussion_r246329142 --- Diff: processing/src/main/java/org/apache/carbondata/processing/merger/CarbonCompactionExecutor.java --- @@ -105,10 +105,15 @@ public

[GitHub] carbondata pull request #3055: [CARBONDATA-3237] Fix presto carbon issues in...

2019-01-09 Thread ajantha-bhat
Github user ajantha-bhat commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3055#discussion_r246319596 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/readers/SliceStreamReader.java --- @@ -95,22 +105,14 @@ public

[GitHub] carbondata pull request #3055: [CARBONDATA-3237] Fix presto carbon issues in...

2019-01-09 Thread ajantha-bhat
Github user ajantha-bhat commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3055#discussion_r246318548 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/readers/SliceStreamReader.java --- @@ -142,5 +144,17 @@ public

[GitHub] carbondata pull request #3059: [HOTFIX][DataLoad]fix task assignment issue u...

2019-01-09 Thread KanakaKumar
Github user KanakaKumar commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3059#discussion_r246317841 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonLoaderUtil.java --- @@ -609,6 +613,9 @@ public static Dictionary

[GitHub] carbondata pull request #3059: [HOTFIX][DataLoad]fix task assignment issue u...

2019-01-09 Thread ndwangsen
Github user ndwangsen commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3059#discussion_r246317595 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonLoaderUtil.java --- @@ -575,19 +575,23 @@ public static Dictionary

[GitHub] carbondata pull request #3059: [HOTFIX][DataLoad]fix task assignment issue u...

2019-01-09 Thread KanakaKumar
Github user KanakaKumar commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3059#discussion_r246311819 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonLoaderUtil.java --- @@ -575,19 +575,23 @@ public static Dictionary

[GitHub] carbondata pull request #3059: [HOTFIX][DataLoad]fix task assignment issue u...

2019-01-09 Thread KanakaKumar
Github user KanakaKumar commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3059#discussion_r246311168 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonLoaderUtil.java --- @@ -609,6 +613,9 @@ public static Dictionary

[GitHub] carbondata pull request #3059: [HOTFIX][DataLoad]fix task assignment issue u...

2019-01-09 Thread KanakaKumar
Github user KanakaKumar commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3059#discussion_r246309331 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonLoaderUtil.java --- @@ -575,19 +575,23 @@ public static Dictionary

[GitHub] carbondata pull request #3059: [HOTFIX][DataLoad]fix task assignment issue u...

2019-01-09 Thread ndwangsen
Github user ndwangsen commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3059#discussion_r246299802 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonLoaderUtil.java --- @@ -1164,4 +1156,35 @@ private static void

[GitHub] carbondata pull request #3059: [HOTFIX][DataLoad]fix task assignment issue u...

2019-01-09 Thread ndwangsen
Github user ndwangsen commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3059#discussion_r246299700 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonLoaderUtil.java --- @@ -609,6 +597,10 @@ public static Dictionary

[GitHub] carbondata pull request #2996: [CARBONDATA-3235] Fix Rename-Fail & Datamap-c...

2019-01-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/2996 ---

[GitHub] carbondata pull request #3032: [CARBONDATA-3210] Merge common method into Ca...

2019-01-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/3032 ---

[GitHub] carbondata pull request #3014: [CARBONDATA-3201] Added load level SORT_SCOPE

2019-01-09 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/3014 ---

[GitHub] carbondata pull request #3059: [HOTFIX][DataLoad]fix task assignment issue u...

2019-01-09 Thread KanakaKumar
Github user KanakaKumar commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3059#discussion_r246286723 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonLoaderUtil.java --- @@ -609,6 +597,10 @@ public static Dictionary

[GitHub] carbondata pull request #3059: [HOTFIX][DataLoad]fix task assignment issue u...

2019-01-08 Thread KanakaKumar
Github user KanakaKumar commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3059#discussion_r246286475 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonLoaderUtil.java --- @@ -1164,4 +1156,35 @@ private static void

[GitHub] carbondata pull request #3054: [CARBONDATA-3232] Add example and doc for all...

2019-01-08 Thread xubo245
Github user xubo245 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3054#discussion_r246280098 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/AlluxioExample.scala --- @@ -28,46 +33,86 @@ import

[GitHub] carbondata pull request #3059: [HOTFIX][DataLoad]fix task assignment issue u...

2019-01-08 Thread ndwangsen
GitHub user ndwangsen opened a pull request: https://github.com/apache/carbondata/pull/3059 [HOTFIX][DataLoad]fix task assignment issue using NODE_MIN_SIZE_FIRST block assignment strategy fix task assignment issue using NODE_MIN_SIZE_FIRST block assignment strategy

[GitHub] carbondata pull request #3055: [CARBONDATA-3237] Fix presto carbon issues in...

2019-01-08 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3055#discussion_r246277431 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/readers/SliceStreamReader.java --- @@ -142,5 +144,17 @@ public

[GitHub] carbondata pull request #3055: [CARBONDATA-3237] Fix presto carbon issues in...

2019-01-08 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3055#discussion_r246277337 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/readers/SliceStreamReader.java --- @@ -95,22 +105,14 @@ public

[GitHub] carbondata pull request #3014: [CARBONDATA-3201] Added load level SORT_SCOPE

2019-01-08 Thread NamanRastogi
Github user NamanRastogi commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3014#discussion_r246274857 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/management/CarbonLoadDataCommand.scala --- @@ -191,10 +191,17

[GitHub] carbondata pull request #3058: [CARBONDATA-3238] Solve StackOverflowError us...

2019-01-08 Thread qiuchenjian
GitHub user qiuchenjian opened a pull request: https://github.com/apache/carbondata/pull/3058 [CARBONDATA-3238] Solve StackOverflowError using MV datamap 【Problem】 An exception or error caused a run to abort. (Using MV) java.lang.StackOverflowError at

[GitHub] carbondata pull request #3057: [Test][CARBONDATA-3238] Solve StackOverflowEr...

2019-01-08 Thread qiuchenjian
Github user qiuchenjian closed the pull request at: https://github.com/apache/carbondata/pull/3057 ---

[GitHub] carbondata pull request #3057: [Test][CARBONDATA-3238] Solve StackOverflowEr...

2019-01-08 Thread qiuchenjian
GitHub user qiuchenjian reopened a pull request: https://github.com/apache/carbondata/pull/3057 [Test][CARBONDATA-3238] Solve StackOverflowError using MV datamap Be sure to do all of the following checklist to help us incorporate your contribution quickly and easily:

[GitHub] carbondata pull request #3057: [Test][CARBONDATA-3238] Solve StackOverflowEr...

2019-01-08 Thread qiuchenjian
Github user qiuchenjian closed the pull request at: https://github.com/apache/carbondata/pull/3057 ---

[GitHub] carbondata pull request #3057: [CARBONDATA-3238] Solve StackOverflowError us...

2019-01-08 Thread qiuchenjian
GitHub user qiuchenjian opened a pull request: https://github.com/apache/carbondata/pull/3057 [CARBONDATA-3238] Solve StackOverflowError using MV datamap 【Problem】 An exception or error caused a run to abort. (Using MV) java.lang.StackOverflowError at

[GitHub] carbondata pull request #3054: [CARBONDATA-3232] Add example and doc for all...

2019-01-08 Thread xubo245
Github user xubo245 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3054#discussion_r246249496 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/AlluxioExample.scala --- @@ -28,46 +33,86 @@ import

[GitHub] carbondata pull request #3054: [CARBONDATA-3232] Add example and doc for all...

2019-01-08 Thread xubo245
Github user xubo245 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3054#discussion_r246249301 --- Diff: docs/alluxio-guide.md --- @@ -0,0 +1,42 @@ + + + +# Presto guide --- End diff -- changed ---

[GitHub] carbondata pull request #3046: [CARBONDATA-3231] Fix OOM exception when dict...

2019-01-08 Thread xuchuanyin
Github user xuchuanyin commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3046#discussion_r246056127 --- Diff: core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java --- @@ -2076,4 +2076,15 @@ private

[GitHub] carbondata pull request #3054: [CARBONDATA-3232] Add example and doc for all...

2019-01-08 Thread xuchuanyin
Github user xuchuanyin commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3054#discussion_r246047276 --- Diff: docs/documentation.md --- @@ -29,15 +29,15 @@ Apache CarbonData is a new big data file format for faster interactive query usi

[GitHub] carbondata pull request #3054: [CARBONDATA-3232] Add example and doc for all...

2019-01-08 Thread xuchuanyin
Github user xuchuanyin commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3054#discussion_r246049322 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/AlluxioExample.scala --- @@ -28,46 +33,86 @@ import

[GitHub] carbondata pull request #3054: [CARBONDATA-3232] Add example and doc for all...

2019-01-08 Thread xuchuanyin
Github user xuchuanyin commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3054#discussion_r246050916 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/AlluxioExample.scala --- @@ -28,46 +33,86 @@ import

[GitHub] carbondata pull request #3054: [CARBONDATA-3232] Add example and doc for all...

2019-01-08 Thread xuchuanyin
Github user xuchuanyin commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3054#discussion_r246044066 --- Diff: docs/alluxio-guide.md --- @@ -0,0 +1,42 @@ + + + +# Presto guide --- End diff -- presto? ---

[GitHub] carbondata pull request #3054: [CARBONDATA-3232] Add example and doc for all...

2019-01-08 Thread xuchuanyin
Github user xuchuanyin commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3054#discussion_r246047576 --- Diff: docs/quick-start-guide.md --- @@ -54,7 +54,8 @@ CarbonData can be integrated with Spark,Presto and Hive Execution Engines. The b ###

[GitHub] carbondata pull request #3032: [CARBONDATA-3210] Merge common method into Ca...

2019-01-08 Thread xiaohui0318
Github user xiaohui0318 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3032#discussion_r246029848 --- Diff: integration/spark2/src/main/scala/org/apache/carbondata/spark/thriftserver/CarbonThriftServer.scala --- @@ -28,12 +28,13 @@ import

[GitHub] carbondata pull request #3032: [CARBONDATA-3210] Merge common method into Ca...

2019-01-08 Thread xiaohui0318
Github user xiaohui0318 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3032#discussion_r246029862 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/S3UsingSDkExample.scala --- @@ -16,28 +16,26 @@ */ package

[GitHub] carbondata pull request #3032: [CARBONDATA-3210] Merge common method into Ca...

2019-01-08 Thread xiaohui0318
Github user xiaohui0318 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3032#discussion_r246029819 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/S3Example.scala --- @@ -18,11 +18,10 @@ package

[GitHub] carbondata pull request #3053: [CARBONDATA-3233]Fix JVM crash issue in snapp...

2019-01-08 Thread qiuchenjian
Github user qiuchenjian commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3053#discussion_r245982727 --- Diff: core/src/main/java/org/apache/carbondata/core/datastore/page/UnsafeFixLengthColumnPage.java --- @@ -369,7 +367,7 @@ public BigDecimal

[GitHub] carbondata pull request #3056: [CARBONDATA-3236] Fix for JVM Crash for inser...

2019-01-08 Thread manishnalla1994
GitHub user manishnalla1994 opened a pull request: https://github.com/apache/carbondata/pull/3056 [CARBONDATA-3236] Fix for JVM Crash for insert into new table from old table Problem: Insert into new table from old table fails with JVM crash. This happened because both the query

[GitHub] carbondata pull request #2996: [CARBONDATA-3235] Fix Rename-Fail & Datamap-c...

2019-01-08 Thread NamanRastogi
Github user NamanRastogi commented on a diff in the pull request: https://github.com/apache/carbondata/pull/2996#discussion_r245925395 --- Diff: integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/schema/CarbonAlterTableRenameCommand.scala --- @@ -165,15

[GitHub] carbondata pull request #3001: [CARBONDATA-3220] Support presto to read stre...

2019-01-08 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3001#discussion_r245916085 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/stream/CarbondataStreamPageSource.java --- @@ -0,0 +1,243 @@ +/* + *

[GitHub] carbondata pull request #3001: [CARBONDATA-3220] Support presto to read stre...

2019-01-08 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3001#discussion_r245916195 --- Diff: integration/presto/src/test/scala/org/apache/carbondata/presto/integrationtest/PrestoAllDataTypeLocalDictTest.scala --- @@ -38,48 +38,16 @@

[GitHub] carbondata pull request #3001: [CARBONDATA-3220] Support presto to read stre...

2019-01-08 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3001#discussion_r245916004 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/readers/BooleanStreamReader.java --- @@ -86,4 +86,15 @@ public

[GitHub] carbondata pull request #3001: [CARBONDATA-3220] Support presto to read stre...

2019-01-08 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3001#discussion_r245915288 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/CarbondataPageSourceProvider.java --- @@ -79,13 +80,31 @@ @Override

[GitHub] carbondata pull request #3001: [CARBONDATA-3220] Support presto to read stre...

2019-01-08 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3001#discussion_r245912249 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/CarbonVectorBatch.java --- @@ -95,6 +95,9 @@ private CarbonColumnVectorImpl

[GitHub] carbondata pull request #3054: [CARBONDATA-3232] Add example and doc for all...

2019-01-08 Thread xubo245
Github user xubo245 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3054#discussion_r245905219 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/util/ExampleUtils.scala --- @@ -30,13 +30,20 @@ object ExampleUtils {

[GitHub] carbondata pull request #3001: [CARBONDATA-3220] Support presto to read stre...

2019-01-07 Thread QiangCai
Github user QiangCai commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3001#discussion_r245903277 --- Diff: integration/presto/src/main/java/org/apache/carbondata/presto/CarbonVectorBatch.java --- @@ -95,6 +95,9 @@ private CarbonColumnVectorImpl

[GitHub] carbondata pull request #3054: [CARBONDATA-3232] Add example and doc for all...

2019-01-07 Thread kevinjmh
Github user kevinjmh commented on a diff in the pull request: https://github.com/apache/carbondata/pull/3054#discussion_r245902021 --- Diff: examples/spark2/src/main/scala/org/apache/carbondata/examples/util/ExampleUtils.scala --- @@ -30,13 +30,20 @@ object ExampleUtils {

[GitHub] carbondata pull request #3024: [CARBONDATA-3230] Add alter test case for dat...

2019-01-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/3024 ---

[GitHub] carbondata pull request #3051: [CARBONDATA-3221] Fix the error of SDK don't ...

2019-01-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/3051 ---

[GitHub] carbondata pull request #2971: [CARBONDATA-3219] Support range partition the...

2019-01-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/2971 ---

  1   2   3   4   5   6   7   8   9   10   >