Github user shardul-cr7 commented on the issue:
https://github.com/apache/carbondata/pull/2850
retest this please
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2869
Build Failed with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9438/
---
Github user KanakaKumar commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2877#discussion_r229579618
--- Diff:
core/src/main/java/org/apache/carbondata/core/metadata/encoder/Encoding.java ---
@@ -57,10 +62,55 @@ public static Encoding valueOf(int or
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2876
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9437/
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/2816
retest this please
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/2807
@ajantha-bhat updated, please check again.
---
Github user xuchuanyin commented on the issue:
https://github.com/apache/carbondata/pull/2879
@kevinjmh OK, no better solutions for that, so current implementation is OK
for me
---
Github user manishgupta88 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2868#discussion_r229578811
--- Diff:
core/src/main/java/org/apache/carbondata/core/util/CarbonUtil.java ---
@@ -300,40 +300,27 @@ public static void deleteFoldersAndFiles(fi
Github user manishgupta88 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2868#discussion_r229578798
--- Diff:
core/src/main/java/org/apache/carbondata/core/util/CarbonUtil.java ---
@@ -300,40 +300,27 @@ public static void deleteFoldersAndFiles(fi
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/2884
[CARBONDATA-3063] Support set and get carbon property in C++ SDK
when user write CarbonData or read CarbonData in C++ SDK, user maybe need
to change or add carbon property to avoid some problem
Github user KanakaKumar commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2877#discussion_r229578569
--- Diff:
core/src/main/java/org/apache/carbondata/core/util/CarbonProperties.java ---
@@ -696,9 +697,6 @@ private void validateCarbonDataFileVersio
Github user KanakaKumar commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2877#discussion_r229578345
--- Diff:
core/src/main/java/org/apache/carbondata/core/metadata/encoder/Encoding.java ---
@@ -57,10 +62,55 @@ public static Encoding valueOf(int or
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2869
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1387/
---
Github user KanakaKumar commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2877#discussion_r229578078
--- Diff:
core/src/main/java/org/apache/carbondata/core/metadata/encoder/Encoding.java ---
@@ -57,10 +62,55 @@ public static Encoding valueOf(int or
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2868#discussion_r229577891
--- Diff:
core/src/main/java/org/apache/carbondata/core/util/CarbonUtil.java ---
@@ -300,40 +300,27 @@ public static void deleteFoldersAndFiles(final
F
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2868#discussion_r229577855
--- Diff:
core/src/main/java/org/apache/carbondata/core/util/CarbonUtil.java ---
@@ -300,40 +300,27 @@ public static void deleteFoldersAndFiles(final
F
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2877
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1385/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2876
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1173/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2877
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1172/
---
Github user manishgupta88 commented on the issue:
https://github.com/apache/carbondata/pull/2876
LGTM
---
Github user kunal642 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2869#discussion_r229563795
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/result/vector/impl/CarbonColumnVectorImpl.java
---
@@ -305,7 +301,7 @@ public void setBlo
Github user kunal642 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2869#discussion_r229563694
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/api/CarbonFileInputFormat.java
---
@@ -145,9 +158,33 @@ public CarbonTable
getOrCreateCar
Github user kunal642 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2869#discussion_r229563660
--- Diff:
store/sdk/src/test/java/org/apache/carbondata/sdk/file/CarbonReaderTest.java ---
@@ -1737,4 +1738,89 @@ public void
testReadNextRowWithProje
Github user kunal642 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2869#discussion_r229563709
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputSplit.java ---
@@ -138,6 +138,19 @@ public CarbonInputSplit(String segmentId, Pa
Github user kunal642 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2869#discussion_r229563684
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/api/CarbonFileInputFormat.java
---
@@ -145,9 +158,33 @@ public CarbonTable
getOrCreateCar
Github user kunal642 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2869#discussion_r229563650
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/util/CarbonVectorizedRecordReader.java
---
@@ -0,0 +1,194 @@
+/*
+ * Licensed to t
Github user kunal642 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2869#discussion_r229563643
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/util/CarbonVectorizedRecordReader.java
---
@@ -0,0 +1,194 @@
+/*
+ * Licensed to t
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2807
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1384/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2807
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9435/
---
xubo245 created CARBONDATA-3063:
---
Summary: Support set carbon property in CSDK
Key: CARBONDATA-3063
URL: https://issues.apache.org/jira/browse/CARBONDATA-3063
Project: CarbonData
Issue Type: Su
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2816
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1383/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2816
Build Failed with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9434/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2807
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1171/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2816
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1170/
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/2869
please fix CI error @kunal642
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/2807
rebase
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/2804
@CI pass, please check again.
---
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2871#discussion_r229535208
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReaderBuilder.java
---
@@ -169,7 +170,7 @@ public CarbonReaderBuilder withHad
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2874#discussion_r229535025
--- Diff: tools/cli/src/main/java/org/apache/carbondata/tool/DataFile.java
---
@@ -500,4 +503,7 @@ private double computePercentage(byte[] data, byte
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2883
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9433/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2883
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1382/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2883
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1169/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2883
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1381/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2883
Build Failed with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9432/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2868
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1380/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2883
Build Failed with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1168/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2876
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1378/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2876
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9429/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2868
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1167/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2877
Build Failed with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9430/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2877
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1379/
---
[
https://issues.apache.org/jira/browse/CARBONDATA-3000?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Kunal Kapoor resolved CARBONDATA-3000.
--
Resolution: Fixed
Fix Version/s: 1.5.1
> Provide C++ interface for writing
Github user asfgit closed the pull request at:
https://github.com/apache/carbondata/pull/2837
---
GitHub user Indhumathi27 opened a pull request:
https://github.com/apache/carbondata/pull/2883
[CARBONDATA-3062] Fix Compatibility issue with cache_level as blocklet
**Why this PR for?**
In case of hybrid store we can have block as well as blocklet schema.
Scenario:
Indhumathi Muthumurugesh created CARBONDATA-3062:
Summary: Fix Compatibility issue with cache_level as blocklet
Key: CARBONDATA-3062
URL: https://issues.apache.org/jira/browse/CARBONDATA-3062
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2877
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1166/
---
Github user manishgupta88 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2868#discussion_r229363551
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/filesystem/LocalCarbonFile.java
---
@@ -141,7 +141,12 @@ public boolean renameT
Github user manishgupta88 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2868#discussion_r229363490
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/filesystem/CarbonFile.java
---
@@ -62,6 +62,11 @@
boolean renameFo
Github user manishgupta88 commented on the issue:
https://github.com/apache/carbondata/pull/2868
> If the table is on S3, will it behave correctly since it does not have
"folder" concept?
I have not changed any existing behavior, so it should work fine
---
Manish Gupta created CARBONDATA-3061:
Summary: Add validation for supported format version and Encoding
type to throw proper exception to the user while reading a file
Key: CARBONDATA-3061
URL: https://issues.
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2876
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1165/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2861
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9428/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2869
Build Failed with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9427/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2861
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1164/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2869
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1163/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2882
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1162/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2882
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1377/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2861
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1376/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2804
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9425/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2869
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1375/
---
Github user kumarvishal09 commented on the issue:
https://github.com/apache/carbondata/pull/2873
LGTM
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2804
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1161/
---
Github user KanakaKumar commented on the issue:
https://github.com/apache/carbondata/pull/2837
LGTM
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2804
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1374/
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/2870
LGTM
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2816
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1160/
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/2867
LGTM
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2873
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9423/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2879
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1159/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2816
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9424/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2882
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9421/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2873
Build Failed with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1372/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2816
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1373/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2879
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9422/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2879
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1371/
---
Github user akashrn5 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2882#discussion_r229300837
--- Diff:
tools/cli/src/main/java/org/apache/carbondata/tool/DataSummary.java ---
@@ -314,23 +312,26 @@ private void printColumnStats(String columnName
Github user NamanRastogi commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2850#discussion_r229299196
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java ---
@@ -114,6 +117,43 @@ public static CarbonReaderBuilder build
Github user ajantha-bhat commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2869#discussion_r229298501
--- Diff:
store/sdk/src/test/java/org/apache/carbondata/sdk/file/CarbonReaderTest.java ---
@@ -1737,4 +1738,89 @@ public void
testReadNextRowWithP
Github user akashrn5 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2882#discussion_r229297954
--- Diff: tools/cli/src/main/java/org/apache/carbondata/tool/DataFile.java
---
@@ -443,7 +444,8 @@ void computePercentage(byte[] shardMin, byte[]
shar
Github user akashrn5 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2861#discussion_r229294323
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/datasources/SparkCarbonTableFormat.scala
---
@@ -175,6 +172,11 @@ with Seri
Github user kunal642 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2869#discussion_r229289006
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/filesystem/LocalCarbonFile.java
---
@@ -178,6 +178,25 @@ public boolean delete() {
Github user kunal642 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2869#discussion_r229288967
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/SafeVariableLengthDimensionDataChunkStore.java
---
@@ -169,7 +
Github user kevinjmh commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2879#discussion_r229287888
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/loading/steps/CarbonRowDataWriterProcessorStepImpl.java
---
@@ -308,7 +312,7 @@ pr
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2804
Build Success with Spark 2.3.1, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/9420/
---
Github user jackylk commented on the issue:
https://github.com/apache/carbondata/pull/2869
Can you change the PR title to be more specific
---
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2861#discussion_r229285778
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/datasources/SparkCarbonTableFormat.scala
---
@@ -175,6 +172,11 @@ with Seria
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2861#discussion_r229285529
--- Diff:
integration/spark-datasource/src/main/scala/org/apache/spark/sql/carbondata/execution/datasources/SparkCarbonFileFormat.scala
---
@@ -121,15
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2861#discussion_r229285477
--- Diff:
integration/spark-common/src/main/scala/org/apache/carbondata/spark/rdd/CarbonRDD.scala
---
@@ -37,6 +37,11 @@ abstract class CarbonRDD[T: Cl
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2861
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/1367/
---
Github user kevinjmh commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2879#discussion_r229285339
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/loading/steps/CarbonRowDataWriterProcessorStepImpl.java
---
@@ -212,7 +212,11 @@ p
1 - 100 of 205 matches
Mail list logo