Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/3048
@KanakaKumar Please review it again.
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3032
Build Success with Spark 2.3.2, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10443/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3032
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2403/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3032
Build Success with Spark 2.3.2, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10441/
---
Kunal Kapoor created CARBONDATA-3231:
Summary: OOM exception is being thrown with varchar columns when
local_dictionary is enabled
Key: CARBONDATA-3231
URL:
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3032
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2187/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2996
Build Failed with Spark 2.3.2, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10440/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3046
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2402/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3046
Build Failed with Spark 2.3.2, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10442/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3046
Build Failed with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2186/
---
Github user kunal642 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3046#discussion_r24497
--- Diff:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
---
@@ -2076,4 +2076,15 @@ private
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3032
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2185/
---
[
https://issues.apache.org/jira/browse/CARBONDATA-3223?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Manish Gupta resolved CARBONDATA-3223.
--
Resolution: Fixed
Fix Version/s: 1.5.3
> Datasize and Indexsize showing 0B
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2996
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2400/
---
Github user asfgit closed the pull request at:
https://github.com/apache/carbondata/pull/3047
---
Github user kunal642 commented on the issue:
https://github.com/apache/carbondata/pull/3046
@xuchuanyin The problem was that when using varchar column with email data
the key for the dictionary map is very huge. When fallback happens the same
data is kept in memory twice, which
Github user Shubh18s commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3010#discussion_r245553671
--- Diff:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
---
@@ -1449,12 +1449,9 @@ private
Github user Shubh18s commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3010#discussion_r245553499
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/preaaggregate/PreAggregateUtil.scala
---
@@ -626,22 +626,29 @@
Github user Shubh18s commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3010#discussion_r245553484
--- Diff:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
---
@@ -1445,16 +1445,10 @@ private
Github user Shubh18s commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3010#discussion_r245553556
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/optimizer/CarbonLateDecodeRule.scala
---
@@ -106,22 +106,29 @@ class
Github user zzcclp commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3032#discussion_r245553139
--- Diff:
examples/spark2/src/main/scala/org/apache/carbondata/examples/S3Example.scala
---
@@ -18,52 +18,50 @@ package org.apache.carbondata.examples
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3048
Build Success with Spark 2.3.2, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10439/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3048
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2399/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2996
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2184/
---
Github user manishgupta88 commented on the issue:
https://github.com/apache/carbondata/pull/3047
LGTM
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3032
Build Success with Spark 2.3.2, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10438/
---
Github user Indhumathi27 commented on the issue:
https://github.com/apache/carbondata/pull/2996
retest this please
---
Github user Shubh18s commented on the issue:
https://github.com/apache/carbondata/pull/3010
Respected Reviewer please Merge this PR.
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2919
Build Success with Spark 2.3.2, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10437/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3032
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2398/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3048
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2183/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3037
Build Success with Spark 2.3.2, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10436/
---
Github user QiangCai commented on the issue:
https://github.com/apache/carbondata/pull/2971
@ravipesala
I agree with you to add it to the table properties.
Even if it becomes the table property, maybe the user also can change it.
right?
Range_column is different from the
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2919
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2396/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3032
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2182/
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/3048
retest this please
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3048
Build Failed with Spark 2.3.2, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10435/
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/3052
@XiaotaoYi Please fix the CI errors.
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3050
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2393/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2919
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2181/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3037
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2395/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3050
Build Success with Spark 2.3.2, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10434/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3048
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2394/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3037
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2180/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3052
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2392/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3052
Build Failed with Spark 2.3.2, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10433/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3048
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2179/
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/3037
retest this please
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/2971
@QiangCai My question how the user can benefit if he chooses a different
range column for each load. I feel range column should be at the table level
not at the load level.
And regarding
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3050
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2178/
---
Github user xuchuanyin commented on the issue:
https://github.com/apache/carbondata/pull/3045
LGTM
---
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3045#discussion_r245534858
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/preaaggregate/PreAggregateTableHelper.scala
---
@@ -110,7
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3050#discussion_r245534776
--- Diff:
datamap/lucene/src/main/java/org/apache/carbondata/datamap/lucene/LuceneFineGrainDataMapFactory.java
---
@@ -57,7 +57,7 @@ public
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/3052
@XiaotaoYi Please optimize the title, like: [CARBONDATA-3227] Fix some
spell errors in the project
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3052
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2177/
---
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3048#discussion_r245534079
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
---
@@ -179,7 +181,8 @@ public CarbonWriterBuilder
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/3050
add to whitelist
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/3052
add to whitelist
---
Github user bbinwang commented on the issue:
https://github.com/apache/carbondata/pull/3050
> @bbinwang Please optimize the title to:[CARBONDATA-3211] Optimize the
documentation
ok
---
Github user KanakaKumar commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3048#discussion_r245532720
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
---
@@ -179,7 +181,8 @@ public CarbonWriterBuilder
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3045
Build Success with Spark 2.3.2, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10432/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3045
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2391/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3045
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2176/
---
Github user shardul-cr7 commented on the issue:
https://github.com/apache/carbondata/pull/3045
retest this please
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3045
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2390/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3045
Build Success with Spark 2.3.2, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10431/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3045
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2175/
---
Github user shardul-cr7 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3045#discussion_r245516216
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/preaaggregate/PreAggregateTableHelper.scala
---
@@ -110,7
Github user shardul-cr7 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3045#discussion_r245516158
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/preaaggregate/PreAggregateTableHelper.scala
---
@@ -110,7
Github user XiaotaoYi commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3052#discussion_r245512840
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
---
@@ -668,9 +668,9 @@ abstract class
Github user kevinjmh commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3045#discussion_r245511058
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/preaaggregate/PreAggregateTableHelper.scala
---
@@ -110,7 +110,29
Github user kevinjmh commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3045#discussion_r245511008
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/preaaggregate/PreAggregateTableHelper.scala
---
@@ -110,7 +110,29
Github user xuchuanyin commented on the issue:
https://github.com/apache/carbondata/pull/3046
Hi @kunal642 ï¼in your PR, the threshold size for storage of the local
dictionary is specified by system (maybe later can be specified by user). But
it will come up with an obvious problem
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3046#discussion_r245510146
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/DecoderBasedFallbackEncoder.java
---
@@ -57,10 +57,7 @@ public
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3046#discussion_r245510098
--- Diff:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
---
@@ -2076,4 +2076,15 @@ private
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3045#discussion_r245509909
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/preaaggregate/PreAggregateTableHelper.scala
---
@@ -110,7
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3023#discussion_r245509625
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonEnv.scala ---
@@ -184,6 +184,9 @@ object CarbonEnv {
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3023#discussion_r245509427
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/events/MergeBloomIndexEventListener.scala
---
@@ -24,59 +24,96 @@ import
Github user qiuchenjian commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3052#discussion_r245505788
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
---
@@ -837,9 +837,9 @@ abstract class
Github user qiuchenjian commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3052#discussion_r245505775
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
---
@@ -821,10 +821,10 @@ abstract class
Github user qiuchenjian commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3052#discussion_r245505760
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
---
@@ -796,10 +796,10 @@ abstract class
Github user qiuchenjian commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3052#discussion_r245505699
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
---
@@ -697,9 +697,9 @@ abstract class
Github user qiuchenjian commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/3052#discussion_r245505691
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
---
@@ -668,9 +668,9 @@ abstract class
Github user qiuchenjian commented on the issue:
https://github.com/apache/carbondata/pull/3052
add to whitelist
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3023
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2389/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3023
Build Success with Spark 2.3.2, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10430/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3023
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/2174/
---
87 matches
Mail list logo