Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r133377209
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/readers/DecimalSliceStreamReader.java
---
@@ -0,0 +1,103 @@
+package
Github user xuchuanyin commented on the issue:
https://github.com/apache/carbondata/pull/1253
A new PR #1261 has been raised to solve this problem in a better way.
---
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 user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r133376989
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/readers/BooleanStreamReader.java
---
@@ -0,0 +1,31 @@
+package
Github user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r133371582
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/expression/ExpressionResult.java
---
@@ -183,7 +183,7 @@ public String getString()
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1236
@steven-qin this pr has resolved this issue with more optimizations #1257
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user sraghunandan commented on the issue:
https://github.com/apache/carbondata/pull/1259
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 user xuchuanyin closed the pull request at:
https://github.com/apache/carbondata/pull/1253
---
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 user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r133370557
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/expression/ExpressionResult.java
---
@@ -183,7 +183,7 @@ public String getString()
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r133372855
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/constants/CarbonCommonConstants.java
---
@@ -0,0 +1,1319 @@
+/*
Github user xuchuanyin commented on the issue:
https://github.com/apache/carbondata/pull/1261
Previous PR #1253 is closed, use this new PR
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
GitHub user xuchuanyin opened a pull request:
https://github.com/apache/carbondata/pull/1261
[CARBONDATA-1373] Enhance update performance by increasing parallelism
# Scenario
Recently I have tested the update feature provided in Carbondata and found
its poor performance.
Github user lionelcao commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1254#discussion_r133364318
--- Diff:
core/src/main/java/org/apache/carbondata/core/keygenerator/directdictionary/timestamp/DateDirectDictionaryGenerator.java
---
@@ -154,14
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1257
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 user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r133375796
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/memory/AggregatedMemoryContext.java
---
@@ -0,0 +1,62 @@
+/*
+ *
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1235
@steven-qin can you please provide reproducable steps?
---
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
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r133375914
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/memory/LocalMemoryContext.java
---
@@ -0,0 +1,40 @@
+/*
+ *
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r133375740
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/memory/AbstractAggregatedMemoryContext.java
---
@@ -0,0 +1,37 @@
+/*
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r133377510
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/readers/DecimalSliceStreamReader.java
---
@@ -0,0 +1,103 @@
+package
Github user mohammadshahidkhan closed the pull request at:
https://github.com/apache/carbondata/pull/1202
---
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
Github user mohammadshahidkhan commented on the issue:
https://github.com/apache/carbondata/pull/1202
not required
---
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 user sraghunandan commented on the issue:
https://github.com/apache/carbondata/pull/1204
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 user anubhav100 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1257#discussion_r133407524
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/constants/CarbonCommonConstants.java
---
@@ -0,0 +1,1319 @@
+/*
---
Github user mohammadshahidkhan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1126#discussion_r133404888
--- Diff:
core/src/main/java/org/apache/carbondata/core/keygenerator/directdictionary/timestamp/DateDirectDictionaryGenerator.java
---
@@
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1234
@linqer how to test this pr can you provide reproducable steps?
---
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 user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1257
SDV Build Failed with Spark 2.1, Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/197/
---
If your project is set up for it, you can reply to this email and have your
reply
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1204
SDV Build Success with Spark 2.1, Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/198/
---
If your project is set up for it, you can reply to this email and have your
reply
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1236
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 user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1236
@anubhav100 i prefer to using steven-qin's pull request, because steven-qin
fix it earlier.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1258
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 user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1260
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 user asfgit closed the pull request at:
https://github.com/apache/carbondata/pull/1252
---
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
[
https://issues.apache.org/jira/browse/CARBONDATA-1372?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Liang Chen resolved CARBONDATA-1372.
Resolution: Fixed
Fix Version/s: 1.2.0
> Fix some errors and update the
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1260
SDV Build Failed with Spark 2.1, Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/201/
---
If your project is set up for it, you can reply to this email and have your
reply
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1236
@chenliang sure he has done a great job
---
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 user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1258
SDV Build Success with Spark 2.1, Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/200/
---
If your project is set up for it, you can reply to this email and have your
reply
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1236
SDV Build Success with Spark 2.1, Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/202/
---
If your project is set up for it, you can reply to this email and have your
reply
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1240#discussion_r133606402
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/encoding/RLECodec.java
---
@@ -0,0 +1,417 @@
+/*
+ * Licensed to the
Github user zzcclp commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1258#discussion_r133619148
--- Diff: docs/partition-guide.md ---
@@ -0,0 +1,124 @@
+
+
+### CarbonData Partition Table Guidance
+This guidance illustrates how to
Github user jackylk closed the pull request at:
https://github.com/apache/carbondata/pull/1249
---
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 user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1231
SDV Build Success with Spark 2.1, Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/206/
---
If your project is set up for it, you can reply to this email and have your
reply
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1231
SDV Build Success with Spark 2.1, Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/203/
---
If your project is set up for it, you can reply to this email and have your
reply
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1240
SDV Build Success with Spark 2.1, Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/204/
---
If your project is set up for it, you can reply to this email and have your
reply
GitHub user jackylk opened a pull request:
https://github.com/apache/carbondata/pull/1262
[BUGFIX] Fix ZERO_BYTE_ARRAY constant not found in codegen
CarbonCommonConstant.ZERO_BYTE_ARRAY is used in codegen, it should not be
deleted. This PR add it back
You can merge this pull
Github user zzcclp commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1258#discussion_r133619308
--- Diff: docs/partition-guide.md ---
@@ -0,0 +1,124 @@
+
+
+### CarbonData Partition Table Guidance
+This guidance illustrates how to
anubhav tarar created CARBONDATA-1385:
-
Summary: Add Test cases for Hive Integration
Key: CARBONDATA-1385
URL: https://issues.apache.org/jira/browse/CARBONDATA-1385
Project: CarbonData
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1231
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 user cenyuhai commented on the issue:
https://github.com/apache/carbondata/pull/1255
1.use hadoop.version instead of 2.6.0
2.use hive.version instead of 1.2.1
3.remove thrift
4.remove zookeeper
5.remove spark-hive and spark-sql
---
If your project is set up for
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1234
@linqer please correct your PR title as per :
https://github.com/apache/carbondata/blob/master/docs/How-to-contribute-to-Apache-CarbonData.md
---
If your project is set up for it, you can
Github user anubhav100 commented on the issue:
https://github.com/apache/carbondata/pull/1236
@steven-qin
1.please correct your PR title as per :
https://github.com/apache/carbondata/blob/master/docs/How-to-contribute-to-Apache-CarbonData.md
2.there is still problem with
Github user QiangCai commented on the issue:
https://github.com/apache/carbondata/pull/1192
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
50 matches
Mail list logo