Github user manishgupta88 commented on the issue:
https://github.com/apache/carbondata/pull/2778
@xuchuanyin .`CarbonLRUCache` instance is one per JVM but cache
implementation is different for different purpose like we have separate cache
provider implementation for blockDataMap
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2790
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/648/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2780
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/842/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2780
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8910/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2782
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/841/
---
Github user ajantha-bhat commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2790#discussion_r221415694
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReaderBuilder.java
---
@@ -115,11 +115,13 @@ public CarbonReaderBuilder
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2782
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8909/
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/2779
My comment : only for 4 parameters , copy the whole
file(CarbonDataSourceScan.scala) for spark 2.3 integration, may not require.
see if can add the judgement for different spark version
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2779#discussion_r221414692
--- Diff:
integration/spark2/src/main/spark2.3/org/apache/spark/sql/execution/strategy/CarbonDataSourceScan.scala
---
@@ -0,0 +1,55 @@
+/*
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2782
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/646/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2780
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/647/
---
Github user xuchuanyin commented on the issue:
https://github.com/apache/carbondata/pull/2778
@manishgupta88 Oh, if it is so, then I think the interfaces of cache are
confusing. Currently we use only one cache instance for all purpose like
mainDataMap, bloomDataMap, dictionary, but
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2780#discussion_r221411423
--- Diff:
store/sdk/src/test/java/org/apache/carbondata/sdk/file/CarbonReaderTest.java ---
@@ -1435,5 +1435,99 @@ public void
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2768#discussion_r221411345
--- Diff:
core/src/main/java/org/apache/carbondata/core/metadata/schema/table/TableSchemaBuilder.java
---
@@ -224,7 +224,7 @@ private ColumnSchema
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2790#discussion_r221411334
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReaderBuilder.java
---
@@ -115,11 +115,13 @@ public CarbonReaderBuilder
Github user QiangCai commented on the issue:
https://github.com/apache/carbondata/pull/2782
@xuchuanyin different test cases use different tables, they are independent.
---
Github user xuchuanyin commented on the issue:
https://github.com/apache/carbondata/pull/2783
@ravipesala @kumarvishal09 We have encountered JVM crash problem.
The dump file complains
Github user zzcclp commented on the issue:
https://github.com/apache/carbondata/pull/2779
@sujith71955 @chenliang613 @ravipesala @jackylk this pr is ready, please
review, thanks.
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2789
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/840/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2789
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8908/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2790
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/839/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2790
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8907/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2789
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/645/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2790
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/644/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2779
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8905/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2787
Build Failed with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8906/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2779
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/643/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2787
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/642/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2788
Build Failed with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8903/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2790
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/641/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2779
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/837/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2790
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/836/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2789
Build Failed with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8902/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2790
Build Failed with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8901/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2787
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/838/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2788
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/834/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2783
Build Failed with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8900/
---
Github user sraghunandan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2788#discussion_r221300203
--- Diff: docs/carbon-as-spark-datasource-guide.md ---
@@ -44,22 +45,20 @@ Carbon table can be created with spark's datasource DDL
syntax as
Github user sraghunandan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2788#discussion_r221295892
--- Diff: docs/carbon-as-spark-datasource-guide.md ---
@@ -15,19 +15,20 @@
limitations under the License.
-->
-# Carbon as
Github user sraghunandan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2788#discussion_r221296016
--- Diff: docs/carbon-as-spark-datasource-guide.md ---
@@ -15,19 +15,20 @@
limitations under the License.
-->
-# Carbon as
Github user sraghunandan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2788#discussion_r221298094
--- Diff: docs/streaming-guide.md ---
@@ -157,7 +157,7 @@ ALTER TABLE streaming_table SET
TBLPROPERTIES('streaming'='true')
At the begin of
Github user sraghunandan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2788#discussion_r221297098
--- Diff: docs/ddl-of-carbondata.md ---
@@ -104,17 +104,18 @@ CarbonData DDL statements are documented here,which
includes:
Following
Github user sraghunandan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2788#discussion_r221296480
--- Diff: docs/ddl-of-carbondata.md ---
@@ -104,17 +104,18 @@ CarbonData DDL statements are documented here,which
includes:
Following
Github user sraghunandan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2788#discussion_r221299726
--- Diff: docs/carbon-as-spark-datasource-guide.md ---
@@ -44,22 +45,20 @@ Carbon table can be created with spark's datasource DDL
syntax as
Github user sraghunandan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2788#discussion_r221295959
--- Diff: docs/carbon-as-spark-datasource-guide.md ---
@@ -15,19 +15,20 @@
limitations under the License.
-->
-# Carbon as
Github user sraghunandan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2788#discussion_r221301565
--- Diff: docs/carbon-as-spark-datasource-guide.md ---
@@ -15,19 +15,20 @@
limitations under the License.
-->
--- End diff --
Github user sraghunandan commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2788#discussion_r221298928
--- Diff: docs/carbon-as-spark-datasource-guide.md ---
@@ -44,22 +45,20 @@ Carbon table can be created with spark's datasource DDL
syntax as
Github user ajantha-bhat commented on the issue:
https://github.com/apache/carbondata/pull/2784
@ravipesala : PR is ready please check
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2789
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/833/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2784
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/832/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2784
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/640/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2787
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/639/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2788
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/638/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2787
Build Failed with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8897/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2789
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/637/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2784
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8899/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2785
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8898/
---
GitHub user ajantha-bhat opened a pull request:
https://github.com/apache/carbondata/pull/2790
[HOTFIX] carbon reader support open another reader without closing previous
reader
[HOTFIX] carbon reader support open another reader without closing previous
reader
[
https://issues.apache.org/jira/browse/CARBONDATA-2989?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Zhichao Zhang reassigned CARBONDATA-2989:
--
Assignee: Zhichao Zhang
> Upgrade spark integration version to 2.3.2
>
Zhichao Zhang created CARBONDATA-2989:
--
Summary: Upgrade spark integration version to 2.3.2
Key: CARBONDATA-2989
URL: https://issues.apache.org/jira/browse/CARBONDATA-2989
Project: CarbonData
Github user ajantha-bhat commented on the issue:
https://github.com/apache/carbondata/pull/2780
LGTM except for minor test case comment by @KanakaKumar .
---
Github user zzcclp commented on the issue:
https://github.com/apache/carbondata/pull/2779
@ravipesala I know how to fix and will fix the tests ASAP.
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2770
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8896/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2786
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/827/
---
Github user kumarvishal09 commented on the issue:
https://github.com/apache/carbondata/pull/2789
LGTM
---
[
https://issues.apache.org/jira/browse/CARBONDATA-2978?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
kumar vishal reassigned CARBONDATA-2978:
Assignee: kumar vishal
> JVM crashes when data inserted from one table to
[
https://issues.apache.org/jira/browse/CARBONDATA-2978?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
kumar vishal resolved CARBONDATA-2978.
--
Resolution: Fixed
> JVM crashes when data inserted from one table to other table
[
https://issues.apache.org/jira/browse/CARBONDATA-2978?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
kumar vishal reassigned CARBONDATA-2978:
Assignee: Ravindra Pesala (was: kumar vishal)
> JVM crashes when data
Github user asfgit closed the pull request at:
https://github.com/apache/carbondata/pull/2773
---
Github user kumarvishal09 commented on the issue:
https://github.com/apache/carbondata/pull/2773
LGTM
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2785
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/829/
---
Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2783#discussion_r221266728
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/model/QueryModel.java ---
@@ -120,7 +121,7 @@
private boolean isFG;
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2783
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/830/
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/2783
@xuchuanyin I don't what is the purpose of doing this? Please elaborate
problem in PR description.
This variable is added to avoid JVM crash when freed multiple times. I am
not getting
Github user zzcclp commented on the issue:
https://github.com/apache/carbondata/pull/2779
@ravipesala can you help me to check why these three test cases fail? It's
about the decimal precision.
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/2779
@zzcclp Please check and fix the tests
---
Github user ajantha-bhat commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2784#discussion_r221261712
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/ColumnPageWrapper.java
---
@@ -302,8 +311,19 @@ public boolean
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2770
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/636/
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/2785
LGTM
---
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2786#discussion_r221260295
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonDatasourceHadoopRelation.scala
---
@@ -80,6 +82,20 @@ case class
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2786#discussion_r221259855
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonDatasourceHadoopRelation.scala
---
@@ -80,6 +82,20 @@ case class
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2787#discussion_r221258708
--- Diff:
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2783
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/635/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2786
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8894/
---
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2784#discussion_r221258211
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/ColumnPageWrapper.java
---
@@ -302,8 +311,19 @@ public boolean
Github user sgururajshetty commented on the issue:
https://github.com/apache/carbondata/pull/2788
@sraghunandan review
---
GitHub user ravipesala opened a pull request:
https://github.com/apache/carbondata/pull/2789
[HOTFIX] Fixed S3 metrics issue.
Problem: When data read from s3 it shows the data read as more than the
size of carbon data total size.
Reason: It happens because carbondata uses
GitHub user sgururajshetty opened a pull request:
https://github.com/apache/carbondata/pull/2788
[Documentation] Readme updated with latest topics and new TOC
> Readme updated with the new structure
> Formatting issue fixed
> Review comments fixed
You can merge this pull
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2770
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/826/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2785
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/634/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2786
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/633/
---
[
https://issues.apache.org/jira/browse/CARBONDATA-2986?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Babulal reassigned CARBONDATA-2986:
---
Assignee: Babulal
> Table Properties are lost when multiple driver concurrently
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2784
Build Failed with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8891/
---
GitHub user kumarvishal09 opened a pull request:
https://github.com/apache/carbondata/pull/2787
[HOTFIX]fixed filter mismatch issue
**Problem:**
Between query is giving wrong result.
**Root cause:**
For timestamp time when filter is given in -mm-dd format instead of
Github user KanakaKumar commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2780#discussion_r221246385
--- Diff:
store/sdk/src/test/java/org/apache/carbondata/sdk/file/CarbonReaderTest.java ---
@@ -1435,5 +1435,99 @@ public void
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2781
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8892/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2781
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/823/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2770
Build Failed with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/8893/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2784
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/822/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2785
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/825/
---
1 - 100 of 169 matches
Mail list logo