Github user QiangCai commented on the issue:
https://github.com/apache/incubator-carbondata/pull/635
@ravipesala
Is it neccessary to limit that the sort_columns should come from dimensions?
If the table need be sorted by a measure, we should use dictionary_include
to add it t
Github user QiangCai commented on the issue:
https://github.com/apache/incubator-carbondata/pull/635
@ravipesala I have listed the tasks.
Better to implement another direct-dictionary encoding for numeric datatype
column. We can remove the dimension and measure concept, and only
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/636
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1051/
---
If your project is set up for it, you can reply to this email and
GitHub user lionelcao opened a pull request:
https://github.com/apache/incubator-carbondata/pull/636
[CARBONDATA-741] Remove useless classes
Remove useless classes
VectorChunkRowIterator
CarbonColumnVectorImpl
You can merge this pull request into a Git repository by run
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/630
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1050/
---
If your project is set up for it, you can reply to this email and
Github user anubhav100 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/630
@jackylk i added a example in CaseClassDataFrameAPIExample,i think its
working correct
case class Employee(name: String, salary: BigDecimal)
val employee = List(Em
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/633
Build Failed with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1049/
---
If your project is set up for it, you can reply to this email and
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/633#discussion_r105079650
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/collector/impl/DictionaryBasedResultCollector.java
---
@@ -103,6 +101,12 @@ p
Github user ravipesala commented on the issue:
https://github.com/apache/incubator-carbondata/pull/635
@QiangCai Please mention what are the tasks you are doing in this PR. It is
better to stick only supporting sort_columns in this PR. Other tasks can be
pushed to other PRs.
---
If
[
https://issues.apache.org/jira/browse/CARBONDATA-743?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ravindra Pesala resolved CARBONDATA-743.
Resolution: Fixed
Fix Version/s: 1.0.1-incubating
> Remove the abundant c
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-carbondata/pull/628
---
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 f
Github user ravipesala commented on the issue:
https://github.com/apache/incubator-carbondata/pull/628
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,
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/633
Build Failed with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1048/
---
If your project is set up for it, you can reply to this email and
Github user lionelcao commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/633#discussion_r105068561
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/collector/impl/DictionaryBasedResultCollector.java
---
@@ -37,6 +34,8 @@
Github user lionelcao commented on the issue:
https://github.com/apache/incubator-carbondata/pull/629
@ravipesala this PR is closed but not merged yet. I can't reopen it.
---
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 QiangCai commented on the issue:
https://github.com/apache/incubator-carbondata/pull/635
@ravipesala good suggestion. Direct dicitonary is better than no
dictioanry. I will add it.
---
If your project is set up for it, you can reply to this email and have your
reply appea
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r105051400
--- Diff: format/src/main/thrift/carbondata_index.thrift ---
@@ -41,6 +41,6 @@ struct IndexHeader{
struct BlockIndex{
1: required i
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/635
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1047/
---
If your project is set up for it, you can reply to this email and
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/631
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1046/
---
If your project is set up for it, you can reply to this email and
Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104971719
--- Diff: format/src/main/thrift/carbondata.thrift ---
@@ -183,11 +183,23 @@ struct FileFooter{
}
/**
+* Footer for i
Github user ravipesala commented on the issue:
https://github.com/apache/incubator-carbondata/pull/628
There is a difference in method `selectFilters` it takes one extra boolean
in deleted file.
---
If your project is set up for it, you can reply to this email and have your
reply app
Github user ravipesala commented on the issue:
https://github.com/apache/incubator-carbondata/pull/620
Data size -> 100 million records
**DDL and Queries for test**
CREATE TABLE perftesta (c1 string,c2 string,c3 string,c4 string,c5
string,c6 bigint,c7 double,c8 int,c9 double,c1
Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104963859
--- Diff: format/src/main/thrift/carbondata.thrift ---
@@ -183,11 +183,23 @@ struct FileFooter{
}
/**
+* Footer for i
Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104963590
--- Diff: format/src/main/thrift/carbondata.thrift ---
@@ -183,11 +183,23 @@ struct FileFooter{
}
/**
+* Footer for i
Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104963505
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/store/writer/v3/CarbonFactDataWriterImplV3.java
---
@@ -319,11 +322
Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104963110
--- Diff: format/src/main/thrift/carbondata_index.thrift ---
@@ -41,6 +41,6 @@ struct IndexHeader{
struct BlockIndex{
1: requ
Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104962645
--- Diff: format/src/main/thrift/carbondata.thrift ---
@@ -183,11 +183,23 @@ struct FileFooter{
}
/**
+* Footer for i
Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104962541
--- Diff: format/src/main/thrift/carbondata.thrift ---
@@ -183,11 +183,23 @@ struct FileFooter{
}
/**
+* Footer for i
Github user ravipesala commented on the issue:
https://github.com/apache/incubator-carbondata/pull/635
@QiangCai I have few doubts.
Why primitive data types are supported as no-dictionary columns in this PR?
It supposed to be direct dictionary.
Why date and timestamp are suppor
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/630
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1045/
---
If your project is set up for it, you can reply to this email and
[
https://issues.apache.org/jira/browse/CARBONDATA-714?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Liang Chen resolved CARBONDATA-714.
---
Resolution: Fixed
Fix Version/s: 1.0.1-incubating
> DOCUMENTATION - How to handle t
[
https://issues.apache.org/jira/browse/CARBONDATA-714?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Liang Chen updated CARBONDATA-714:
--
Summary: DOCUMENTATION - How to handle the bad records (was: DOCUMENTATION
- How to handle
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/614
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 s
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-carbondata/pull/614
---
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 f
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/332
Build Failed with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1044/
---
If your project is set up for it, you can reply to this email and
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/332
Build Failed with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1043/
---
If your project is set up for it, you can reply to this email and
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/634
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1042/
---
If your project is set up for it, you can reply to this email and
GitHub user QiangCai opened a pull request:
https://github.com/apache/incubator-carbondata/pull/635
[WIP]support SORT_COLUMNS
1. create table with sort_columns
e.g. tblproperties('sort_columns' = 'col7,col3')
The columns of sort_columsn will be at the begin of all colu
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/634#discussion_r104926113
--- Diff:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
---
@@ -383,7 +383,7 @@
*/
public
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/634#discussion_r104925662
--- Diff:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
---
@@ -383,7 +383,7 @@
*/
p
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/635
Build Failed with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1041/
---
If your project is set up for it, you can reply to this email and
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/630
Can you add a testcase to write datafame containing a decimal column and
query it? I think it needs more implementation for load and read
---
If your project is set up for it, you can
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/634#discussion_r104919325
--- Diff:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
---
@@ -392,7 +392,7 @@
*/
public
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/634#discussion_r104919217
--- Diff:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
---
@@ -383,7 +383,7 @@
*/
public
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/634
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1040/
---
If your project is set up for it, you can reply to this email and
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/633#discussion_r104916295
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/collector/impl/DictionaryBasedResultCollector.java
---
@@ -37,6 +34,8 @@
im
GitHub user chenliang613 opened a pull request:
https://github.com/apache/incubator-carbondata/pull/634
[CARBONDATA-753] Fix Date and Timestamp format issues
Fix Date and Timestamp format issues:
1.Optimize the description of CARBON_TIMESTAMP_FORMAT&CARBON_DATE_FORMAT
in Carbon
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104915516
--- Diff: format/src/main/thrift/carbondata.thrift ---
@@ -183,11 +183,23 @@ struct FileFooter{
}
/**
+* Footer for indexed
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104915366
--- Diff: format/src/main/thrift/carbondata.thrift ---
@@ -183,11 +183,23 @@ struct FileFooter{
}
/**
+* Footer for indexed
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104915387
--- Diff: format/src/main/thrift/carbondata.thrift ---
@@ -183,11 +183,23 @@ struct FileFooter{
}
/**
+* Footer for indexed
[
https://issues.apache.org/jira/browse/CARBONDATA-753?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Liang Chen updated CARBONDATA-753:
--
Description:
Fix Date and Timestamp format issues:
1.Optimize the description of CARBON_TIME
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104914699
--- Diff: format/src/main/thrift/carbondata.thrift ---
@@ -183,11 +183,23 @@ struct FileFooter{
}
/**
+* Footer for indexed
[
https://issues.apache.org/jira/browse/CARBONDATA-753?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Liang Chen updated CARBONDATA-753:
--
Description:
Fix Date and Timestamp format issues:
1.Optimize the description of CARBON_TIME
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104914302
--- Diff: format/src/main/thrift/carbondata.thrift ---
@@ -183,11 +183,23 @@ struct FileFooter{
}
/**
+* Footer for indexed
Liang Chen created CARBONDATA-753:
-
Summary: Fix Date and Timestamp format issues
Key: CARBONDATA-753
URL: https://issues.apache.org/jira/browse/CARBONDATA-753
Project: CarbonData
Issue Type:
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/631
Build Failed with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1039/
---
If your project is set up for it, you can reply to this email and
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/631
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1038/
---
If your project is set up for it, you can reply to this email and
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104910765
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/store/writer/v3/CarbonFactDataWriterImplV3.java
---
@@ -319,11 +322,12 @@
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104910137
--- Diff: format/src/main/thrift/carbondata.thrift ---
@@ -183,11 +183,23 @@ struct FileFooter{
}
/**
+* Footer for indexed
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104910022
--- Diff: format/src/main/thrift/carbondata_index.thrift ---
@@ -41,6 +41,6 @@ struct IndexHeader{
struct BlockIndex{
1: required i
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104909522
--- Diff: format/src/main/thrift/carbondata.thrift ---
@@ -183,11 +183,23 @@ struct FileFooter{
}
/**
+* Footer for indexed
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104908630
--- Diff: format/src/main/thrift/carbondata.thrift ---
@@ -183,11 +183,23 @@ struct FileFooter{
}
/**
+* Footer for indexed
Github user kumarvishal09 closed the pull request at:
https://github.com/apache/incubator-carbondata/pull/569
---
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 i
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104907969
--- Diff:
core/src/main/java/org/apache/carbondata/core/reader/CarbonHeaderReader.java ---
@@ -0,0 +1,75 @@
+/*
+ * Licensed to the Ap
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104907859
--- Diff:
core/src/main/java/org/apache/carbondata/core/reader/CarbonHeaderReader.java ---
@@ -0,0 +1,75 @@
+/*
+ * Licensed to the Ap
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104907778
--- Diff:
core/src/main/java/org/apache/carbondata/core/reader/CarbonHeaderReader.java ---
@@ -0,0 +1,75 @@
+/*
+ * Licensed to the Ap
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/624
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1037/
---
If your project is set up for it, you can reply to this email and
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104905629
--- Diff:
core/src/main/java/org/apache/carbondata/core/reader/CarbonFooterReaderV3.java
---
@@ -0,0 +1,76 @@
+/*
+ * Licensed to the
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104905047
--- Diff:
core/src/main/java/org/apache/carbondata/core/reader/CarbonFooterReaderV3.java
---
@@ -0,0 +1,76 @@
+/*
+ * Licensed to the
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/618#discussion_r104903081
--- Diff:
integration/spark2/src/test/scala/org/apache/spark/SparkCommandSuite.scala ---
@@ -0,0 +1,38 @@
+/*
+ * Licensed to the Apac
[
https://issues.apache.org/jira/browse/CARBONDATA-740?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ravindra Pesala resolved CARBONDATA-740.
Resolution: Fixed
Fix Version/s: 1.0.1-incubating
> Add logger for rows p
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-carbondata/pull/629
---
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 f
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/631
Build Failed with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1036/
---
If your project is set up for it, you can reply to this email and
Github user ravipesala commented on the issue:
https://github.com/apache/incubator-carbondata/pull/629
LGTM Thanks for working on this.
---
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 fe
[
https://issues.apache.org/jira/browse/CARBONDATA-746?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ravindra Pesala resolved CARBONDATA-746.
Resolution: Fixed
> Support spark-sql CLI for spark2.1 carbon integration
>
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-carbondata/pull/623
---
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 f
Github user ravipesala commented on the issue:
https://github.com/apache/incubator-carbondata/pull/623
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,
anubhav tarar created CARBONDATA-752:
Summary: creating complex type gives exception
Key: CARBONDATA-752
URL: https://issues.apache.org/jira/browse/CARBONDATA-752
Project: CarbonData
Issu
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/631
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1035/
---
If your project is set up for it, you can reply to this email and
Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/631#discussion_r104888546
--- Diff: format/src/main/thrift/carbondata.thrift ---
@@ -183,11 +183,23 @@ struct FileFooter{
}
/**
+* Footer for i
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/622
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1034/
---
If your project is set up for it, you can reply to this email and
[
https://issues.apache.org/jira/browse/CARBONDATA-739?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Cao, Lionel reassigned CARBONDATA-739:
--
Assignee: Cao, Lionel (was: Liang Chen)
> Avoid creating multiple instances of Dir
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/633
Build Failed with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1033/
---
If your project is set up for it, you can reply to this email and
GitHub user lionelcao opened a pull request:
https://github.com/apache/incubator-carbondata/pull/633
[CARBONDATA-739] Avoid creating multipul instance of
directDictionaryGenerator
Avoid creating multipul instance of directDictionaryGenerator
You can merge this pull request into a G
84 matches
Mail list logo