Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062627
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/parser/CarbonSparkSqlParser.scala
---
@@ -18,16 +18,17 @@ package org.apache.spark.
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062611
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/parser/CarbonSpark2SqlParser.scala
---
@@ -177,7 +213,24 @@ class CarbonSpark2SqlPa
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062519
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/parser/CarbonSpark2SqlParser.scala
---
@@ -17,19 +17,23 @@
package org.a
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062511
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/optimizer/CarbonLateDecodeRule.scala
---
@@ -38,7 +38,6 @@ import org.apache.carbon
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062493
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonRelation.scala
---
@@ -172,7 +172,13 @@ case class CarbonRelation(
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062473
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonHiveMetaStore.scala
---
@@ -153,8 +153,11 @@ class CarbonHiveMetaStore e
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062458
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonFileMetastore.scala
---
@@ -115,18 +121,37 @@ class CarbonFileMetastore
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062453
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonFileMetastore.scala
---
@@ -115,18 +121,37 @@ class CarbonFileMetastore
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062424
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonFileMetastore.scala
---
@@ -115,18 +121,37 @@ class CarbonFileMetastore
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062410
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonFileMetastore.scala
---
@@ -115,18 +121,37 @@ class CarbonFileMetastore
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062388
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonAnalysisRules.scala
---
@@ -201,8 +237,10 @@ case class CarbonIUDAnalysi
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062364
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonAnalysisRules.scala
---
@@ -143,52 +168,63 @@ case class CarbonIUDAnalys
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062354
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonAnalysisRules.scala
---
@@ -143,52 +168,63 @@ case class CarbonIUDAnalys
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062338
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonAnalysisRules.scala
---
@@ -143,52 +168,63 @@ case class CarbonIUDAnalys
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1291
SDV Build Success , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/1878/
---
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062310
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonAnalysisRules.scala
---
@@ -143,52 +246,250 @@ case class CarbonIUDAnaly
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062213
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonAnalysisRules.scala
---
@@ -95,12 +107,23 @@ case class CarbonIUDAnalysi
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062196
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonAnalysisRules.scala
---
@@ -95,12 +107,23 @@ case class CarbonIUDAnalysi
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062164
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonAnalysisRules.scala
---
@@ -95,12 +107,23 @@ case class CarbonIUDAnalysi
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062076
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonAnalysisRules.scala
---
@@ -58,6 +57,8 @@ object CarbonPreInsertionCasts
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062059
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonAnalysisRules.scala
---
@@ -24,17 +24,15 @@ import
org.apache.spark.sql
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062031
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/strategy/DDLStrategy.scala
---
@@ -155,6 +161,14 @@ class DDLStrategy(spa
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153062036
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/strategy/DDLStrategy.scala
---
@@ -155,6 +161,14 @@ class DDLStrategy(spa
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1291
Build Failed with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1461/
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1291
SDV Build Fail , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/1877/
---
Github user cenyuhai commented on the issue:
https://github.com/apache/carbondata/pull/1291
@chenliang613 please review this pr
---
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061413
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/strategy/DDLStrategy.scala
---
@@ -26,13 +26,19 @@ import
org.apache.spa
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061418
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/strategy/DDLStrategy.scala
---
@@ -26,13 +26,19 @@ import
org.apache.spa
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061398
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/strategy/CarbonLateDecodeStrategy.scala
---
@@ -42,6 +43,7 @@ import
org
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061370
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/schema/AlterTableDropColumnCommand.scala
---
@@ -120,10 +121,7 @@
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061350
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/schema/AlterTableDropColumnCommand.scala
---
@@ -20,9 +20,10 @@ p
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061341
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/schema/AlterTableDataTypeChangeCommand.scala
---
@@ -94,10 +95,7
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061334
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/schema/AlterTableDataTypeChangeCommand.scala
---
@@ -19,9 +19,10
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061299
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/CastExpressionOptimization.scala
---
@@ -21,12 +21,14 @@ import java.text
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061284
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonSession.scala ---
@@ -43,14 +43,15 @@ class CarbonSession(@transient val sc: S
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061170
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonSession.scala ---
@@ -19,10 +19,10 @@ package org.apache.spark.sql
import
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061160
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonExpressions.scala
---
@@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apach
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061155
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonExpressions.scala
---
@@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apach
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061142
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonExpressions.scala
---
@@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apach
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061126
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonExpressions.scala
---
@@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apach
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061102
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonEnv.scala ---
@@ -22,6 +22,7 @@ import java.util.concurrent.ConcurrentHashMap
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061073
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/util/CarbonReflectionUtils.scala
---
@@ -0,0 +1,90 @@
+/*
+ * Licensed to
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1560
SDV Build Success , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/1876/
---
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061021
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/util/CarbonReflectionUtils.scala
---
@@ -0,0 +1,90 @@
+/*
+ * Licensed to
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061030
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/util/CarbonReflectionUtils.scala
---
@@ -0,0 +1,90 @@
+/*
+ * Licensed to
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153061017
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/util/CarbonReflectionUtils.scala
---
@@ -0,0 +1,90 @@
+/*
+ * Licensed to
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153060970
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/optimizer/CarbonDecoderOptimizerHelper.scala
---
@@ -84,7 +98,16 @@ class Car
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1469#discussion_r153060899
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/optimizer/CarbonDecoderOptimizerHelper.scala
---
@@ -59,6 +62,17 @@ case clas
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1560
Build Failed with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1459/
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1510
SDV Build Fail , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/1875/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1510
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1458/
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1560
SDV Build Fail , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/1874/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1469
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1457/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1560
Build Failed with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1456/
---
Github user sounakr commented on the issue:
https://github.com/apache/carbondata/pull/1469
Retest this please
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1469
Build Failed with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1455/
---
Github user ManoharVanam commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1560#discussion_r153059130
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/impl/DFSFileHolderImpl.java
---
@@ -35,7 +35,7 @@
/**
* cache to
Github user ManoharVanam commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1560#discussion_r153059129
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/hive/CarbonFileMetastore.scala
---
@@ -353,14 +355,14 @@ class CarbonFileMetastor
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1469
SDV Build Fail , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/1873/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1469
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1454/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1503
Build Failed with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1453/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1567
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1452/
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1503
SDV Build Fail , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/1872/
---
Github user sehriff closed the pull request at:
https://github.com/apache/carbondata/pull/1503
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1469
SDV Build Fail , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/1871/
---
Github user akashrn5 commented on the issue:
https://github.com/apache/carbondata/pull/1567
retest this please
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1469
SDV Build Fail , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/1870/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1469
Build Failed with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1451/
---
Github user sehriff closed the pull request at:
https://github.com/apache/carbondata/pull/1503
---
GitHub user sehriff reopened a pull request:
https://github.com/apache/carbondata/pull/1503
[CARBONDATA-1730] Support skip.header.line.count option for Hive Table in
Carbon
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1510
SDV Build Fail , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/1869/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1510
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1450/
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1510
SDV Build Fail , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/1868/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1510
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1449/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1469
Build Failed with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1448/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1567
Build Failed with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1447/
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1469
SDV Build Fail , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/1867/
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/1567
SDV Build Success , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/1866/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1567
Build Failed with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1446/
---
Akash R Nilugal created CARBONDATA-1809:
---
Summary: Add Create Table Event
Key: CARBONDATA-1809
URL: https://issues.apache.org/jira/browse/CARBONDATA-1809
Project: CarbonData
Issue Type:
GitHub user akashrn5 opened a pull request:
https://github.com/apache/carbondata/pull/1567
[CARBONDATA-1809]Add create table event
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:
- [*] Any interfaces changed?
Github user jackylk commented on the issue:
https://github.com/apache/carbondata/pull/1496
LGTM
---
Github user jackylk commented on the issue:
https://github.com/apache/carbondata/pull/1460
@LiShuMing please rebase
---
[
https://issues.apache.org/jira/browse/CARBONDATA-1778?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jacky Li resolved CARBONDATA-1778.
--
Resolution: Fixed
Fix Version/s: 1.3.0
> Support clean garbage segments for all
> --
Github user asfgit closed the pull request at:
https://github.com/apache/carbondata/pull/1537
---
Github user jackylk commented on the issue:
https://github.com/apache/carbondata/pull/1537
LGTM
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/1557
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1445/
---
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1523#discussion_r153040465
--- Diff:
integration/spark2/src/test/scala/org/apache/carbondata/spark/testsuite/booleantype/compress/BooleanFile.scala
---
@@ -0,0 +1,59 @@
+/*
Github user jackylk commented on the issue:
https://github.com/apache/carbondata/pull/1540
I think there are more place has column group information, it is better to
remove all of them in one PR
---
Github user jackylk commented on the issue:
https://github.com/apache/carbondata/pull/1557
Please add description for this PR
@dhatchayani
---
90 matches
Mail list logo