Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r108028962
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/MapredCarbonOutputFormat.java
---
@@ -0,0 +1,49 @@
+/*
+ * Licen
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/696
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1331/
---
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/697
LGTM, except few style issues raised by david, please fix them
---
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/incubator-carbondata/pull/696
add to whitelist
---
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 an
Github user cenyuhai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r108026227
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/MapredCarbonOutputFormat.java
---
@@ -0,0 +1,49 @@
+/*
+ * Licen
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/691
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1330/
---
If your project is set up for it, you can reply to this email and
Github user QiangCai commented on the issue:
https://github.com/apache/incubator-carbondata/pull/691
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 an
Github user QiangCai commented on the issue:
https://github.com/apache/incubator-carbondata/pull/691
Looks good.
---
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 wishe
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/694
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1329/
---
If your project is set up for it, you can reply to this email and
Github user QiangCai commented on the issue:
https://github.com/apache/incubator-carbondata/pull/694
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 an
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/697#discussion_r107919139
--- Diff:
integration/spark-common/src/main/scala/org/apache/carbondata/spark/CarbonFilters.scala
---
@@ -111,16 +111,24 @@ object CarbonFil
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/697#discussion_r107918771
--- Diff:
integration/spark-common/src/main/scala/org/apache/carbondata/spark/CarbonFilters.scala
---
@@ -111,16 +111,24 @@ object CarbonFil
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/697#discussion_r107920260
--- Diff:
integration/spark-common/src/main/scala/org/apache/carbondata/spark/CarbonFilters.scala
---
@@ -111,16 +111,24 @@ object CarbonFil
Github user QiangCai commented on the issue:
https://github.com/apache/incubator-carbondata/pull/696
@watermen
It is unnecessary to store carbondata file path in carbonindex file.
During btree building, just use carbondata file name to sort
tableblockinfos.
please check Ca
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r107893281
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/MapredCarbonInputFormat.java
---
@@ -0,0 +1,99 @@
+/*
+ * Licens
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r107862312
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/CarbonArrayInspector.java
---
@@ -0,0 +1,191 @@
+/*
+ * Licensed
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r107886267
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/CarbonHiveSerDe.java
---
@@ -0,0 +1,232 @@
+/*
+ * Licensed to t
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r107872055
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/CarbonHiveRecordReader.java
---
@@ -0,0 +1,249 @@
+/*
+ * Licens
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r107875522
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/CarbonHiveRecordReader.java
---
@@ -0,0 +1,249 @@
+/*
+ * Licens
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r107863523
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/CarbonArrayInspector.java
---
@@ -0,0 +1,191 @@
+/*
+ * Licensed
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r107894484
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/MapredCarbonInputFormat.java
---
@@ -0,0 +1,99 @@
+/*
+ * Licens
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r107881262
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/CarbonHiveRecordReader.java
---
@@ -0,0 +1,249 @@
+/*
+ * Licens
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r107862824
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/CarbonArrayInspector.java
---
@@ -0,0 +1,191 @@
+/*
+ * Licensed
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r107863410
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/CarbonArrayInspector.java
---
@@ -0,0 +1,191 @@
+/*
+ * Licensed
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r107873741
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/MapredCarbonInputFormat.java
---
@@ -0,0 +1,99 @@
+/*
+ * Licens
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r107890848
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/MapredCarbonOutputFormat.java
---
@@ -0,0 +1,49 @@
+/*
+ * Licen
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r107863922
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/CarbonHiveInputSplit.java
---
@@ -0,0 +1,290 @@
+/*
+ * Licensed
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/672#discussion_r107863484
--- Diff:
integration/hive/src/main/java/org/apache/carbondata/hive/CarbonArrayInspector.java
---
@@ -0,0 +1,191 @@
+/*
+ * Licensed
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/695
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1328/
---
If your project is set up for it, you can reply to this email and
Vinod Rohilla created CARBONDATA-819:
Summary: Getting error while create table with long data type
Key: CARBONDATA-819
URL: https://issues.apache.org/jira/browse/CARBONDATA-819
Project: CarbonData
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/697
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1327/
---
If your project is set up for it, you can reply to this email and
Github user bhavya411 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/616
@ravipesala Created a New PR #697 for the fixes that were suggested , added
a Unit test class to make sure it runs everytime with the build and removed the
example
---
If your proje
GitHub user bhavya411 opened a pull request:
https://github.com/apache/incubator-carbondata/pull/697
[CARBONDATA-708] Fixed Between and Logical AND issue
- Change CarbonFilters.scala to fix BETWEEN and Logical AND operator
- Add Unit Test Cases
- Tested on Spark 1.6 and 2.1
[
https://issues.apache.org/jira/browse/CARBONDATA-813?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Liang Chen resolved CARBONDATA-813.
---
Resolution: Fixed
Fix Version/s: 1.2.0
> Fix pom issues and add the correct depende
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-carbondata/pull/693
---
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/693
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 kumarvishal09 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/695
Please add test case for failure scenario
---
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
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/652
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1326/
---
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/652
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1325/
---
If your project is set up for it, you can reply to this email and
Github user rahulforallp commented on the issue:
https://github.com/apache/incubator-carbondata/pull/652
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
enable
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/652
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1324/
---
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/696
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1323/
---
If your project is set up for it, you can reply to this email and
[
https://issues.apache.org/jira/browse/CARBONDATA-818?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Yadong Qi updated CARBONDATA-818:
-
Description:
The file_name stored in carbonindex is a local path which used on executor as
te
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/689
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1322/
---
If your project is set up for it, you can reply to this email and
GitHub user watermen opened a pull request:
https://github.com/apache/incubator-carbondata/pull/696
[CARBONDATA-818] Make the file_name in carbonindex exactly
The file_name stored in carbonindex is a local path which used on executor
as temp dir
```
/tmp/6937581525189542/0
Github user ravipesala commented on the issue:
https://github.com/apache/incubator-carbondata/pull/689
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
[
https://issues.apache.org/jira/browse/CARBONDATA-818?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Yadong Qi updated CARBONDATA-818:
-
Description: The file_name stored in carbonindex is a local path which used
on executor as tem
[
https://issues.apache.org/jira/browse/CARBONDATA-818?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Yadong Qi updated CARBONDATA-818:
-
Summary: The file_name stored in carbonindex is wrong (was: The file)
> The file_name stored
Yadong Qi created CARBONDATA-818:
Summary: The file
Key: CARBONDATA-818
URL: https://issues.apache.org/jira/browse/CARBONDATA-818
Project: CarbonData
Issue Type: Bug
Reporter: Yad
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/694
Build Failed with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/1321/
---
If your project is set up for it, you can reply to this email and
50 matches
Mail list logo