vikramahuja1001 opened a new pull request #3873:
URL: https://github.com/apache/carbondata/pull/3873
### Why is this PR needed?
### What changes were proposed in this PR?
### Does this PR introduce any user interface change?
- No
- Yes. (please explain
QiangCai opened a new pull request #3872:
URL: https://github.com/apache/carbondata/pull/3872
### Why is this PR needed?
There are java files under the scala source directory
### What changes were proposed in this PR?
1. Move java files into the java source directory
2.
CarbonDataQA1 commented on pull request #3774:
URL: https://github.com/apache/carbondata/pull/3774#issuecomment-665593991
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
ajantha-bhat commented on a change in pull request #3773:
URL: https://github.com/apache/carbondata/pull/3773#discussion_r462270452
##
File path:
integration/presto/src/main/java/org/apache/carbondata/presto/PrestoCarbonVectorizedRecordReader.java
##
@@ -176,8 +191,16 @@ priva
ajantha-bhat opened a new pull request #3871:
URL: https://github.com/apache/carbondata/pull/3871
### Why is this PR needed?
Auto compaction/minor compaction was happening multiple times for same
segments.
Executor (for merge index and merge data files) segmentfile write failur
CarbonDataQA1 commented on pull request #3873:
URL: https://github.com/apache/carbondata/pull/3873#issuecomment-665651226
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
CarbonDataQA1 commented on pull request #3865:
URL: https://github.com/apache/carbondata/pull/3865#issuecomment-665611488
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
vikramahuja1001 commented on pull request #3873:
URL: https://github.com/apache/carbondata/pull/3873#issuecomment-665784554
retest this please
This is an automated message from the Apache Git Service.
To respond to the messag
CarbonDataQA1 commented on pull request #3871:
URL: https://github.com/apache/carbondata/pull/3871#issuecomment-665517022
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
ShreelekhyaG commented on a change in pull request #3774:
URL: https://github.com/apache/carbondata/pull/3774#discussion_r462128423
##
File path:
integration/spark/src/main/scala/org/apache/spark/sql/secondaryindex/command/SICreationCommand.scala
##
@@ -210,6 +210,14 @@ privat
CarbonDataQA1 commented on pull request #3872:
URL: https://github.com/apache/carbondata/pull/3872#issuecomment-665604813
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
CarbonDataQA1 commented on pull request #3869:
URL: https://github.com/apache/carbondata/pull/3869#issuecomment-665577522
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
Indhumathi27 opened a new pull request #3874:
URL: https://github.com/apache/carbondata/pull/3874
### Why is this PR needed?
On data load to SI with date type, dictionary values is loaded from
factToIndexDictColumnMapping instead of getting from wrapper.
### What changes were
Indhumathi Muthumurugesh created CARBONDATA-3931:
Summary: Secondary index with index column as DateType gives wrong
results
Key: CARBONDATA-3931
URL: https://issues.apache.org/jira/browse/CARBONDA
akkio-97 opened a new pull request #3866:
URL: https://github.com/apache/carbondata/pull/3866
### Why is this PR needed?
Data load fails because the spark-shell code uses carbonSessions instead of
carbonExtensions.
### What changes were proposed in this PR?
Have made use of
akkio-97 closed pull request #3859:
URL: https://github.com/apache/carbondata/pull/3859
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go
CarbonDataQA1 commented on pull request #3868:
URL: https://github.com/apache/carbondata/pull/3868#issuecomment-665182981
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
CarbonDataQA1 commented on pull request #3856:
URL: https://github.com/apache/carbondata/pull/3856#issuecomment-665195269
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
akashrn5 commented on pull request #3857:
URL: https://github.com/apache/carbondata/pull/3857#issuecomment-664777049
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
asfgit closed pull request #3864:
URL: https://github.com/apache/carbondata/pull/3864
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go t
CarbonDataQA1 commented on pull request #3773:
URL: https://github.com/apache/carbondata/pull/3773#issuecomment-665065271
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
kevinjmh commented on a change in pull request #3868:
URL: https://github.com/apache/carbondata/pull/3868#discussion_r461986247
##
File path:
core/src/main/java/org/apache/carbondata/core/datastore/page/SafeFixLengthColumnPage.java
##
@@ -283,8 +283,8 @@ public BigDecimal getD
CarbonDataQA1 commented on pull request #3819:
URL: https://github.com/apache/carbondata/pull/3819#issuecomment-665015972
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
QiangCai opened a new pull request #3868:
URL: https://github.com/apache/carbondata/pull/3868
### Why is this PR needed?
1. Redundant 'if' statement
2. Array access many times in for loop
3. Repeated switch branch
4. Use StringBuffer
### What changes were proposed in th
Karan980 commented on pull request #3857:
URL: https://github.com/apache/carbondata/pull/3857#issuecomment-664850764
@VenuReddy2103 Done
This is an automated message from the Apache Git Service.
To respond to the message, ple
CarbonDataQA1 commented on pull request #3870:
URL: https://github.com/apache/carbondata/pull/3870#issuecomment-665433865
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
nihal0107 commented on a change in pull request #3819:
URL: https://github.com/apache/carbondata/pull/3819#discussion_r461474532
##
File path:
sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/AvroCarbonWriter.java
##
@@ -823,6 +834,30 @@ public void write(Object object) th
ajantha-bhat commented on pull request #3773:
URL: https://github.com/apache/carbondata/pull/3773#issuecomment-665473589
@akkio-97 : Also change description to presto supporting array columns
reading
This is an automated mes
niuge01 commented on pull request #3864:
URL: https://github.com/apache/carbondata/pull/3864#issuecomment-664720572
LGTM
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
QiangCai opened a new pull request #3870:
URL: https://github.com/apache/carbondata/pull/3870
### Why is this PR needed?
MVExample is throwing DataLoadingException because the path of the input
file is wrong.
### What changes were proposed in this PR?
1. Correct the path of
CarbonDataQA1 commented on pull request #3789:
URL: https://github.com/apache/carbondata/pull/3789#issuecomment-665169330
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
VenuReddy2103 commented on a change in pull request #3774:
URL: https://github.com/apache/carbondata/pull/3774#discussion_r461692563
##
File path:
integration/spark/src/main/scala/org/apache/spark/sql/secondaryindex/command/SICreationCommand.scala
##
@@ -210,6 +210,14 @@ priva
CarbonDataQA1 commented on pull request #3867:
URL: https://github.com/apache/carbondata/pull/3867#issuecomment-665157764
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
CarbonDataQA1 commented on pull request #3865:
URL: https://github.com/apache/carbondata/pull/3865#issuecomment-665003765
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
CarbonDataQA1 commented on pull request #3774:
URL: https://github.com/apache/carbondata/pull/3774#issuecomment-664592745
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
CarbonDataQA1 commented on pull request #3787:
URL: https://github.com/apache/carbondata/pull/3787#issuecomment-665125549
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
VenuReddy2103 commented on pull request #3857:
URL: https://github.com/apache/carbondata/pull/3857#issuecomment-665140464
LGTM
This is an automated message from the Apache Git Service.
To respond to the message, please log on
Indhumathi27 commented on a change in pull request #3773:
URL: https://github.com/apache/carbondata/pull/3773#discussion_r461655861
##
File path:
core/src/main/java/org/apache/carbondata/core/scan/collector/impl/DictionaryBasedVectorResultCollector.java
##
@@ -98,6 +98,14 @@ v
CarbonDataQA1 commented on pull request #3857:
URL: https://github.com/apache/carbondata/pull/3857#issuecomment-664827883
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
akkio-97 commented on pull request #3866:
URL: https://github.com/apache/carbondata/pull/3866#issuecomment-665281574
retest this please
This is an automated message from the Apache Git Service.
To respond to the message,
Indhumathi27 commented on pull request #3848:
URL: https://github.com/apache/carbondata/pull/3848#issuecomment-664772987
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
QiangCai opened a new pull request #3867:
URL: https://github.com/apache/carbondata/pull/3867
### Why is this PR needed?
There are many typos in carbondata-spark module.
### What changes were proposed in this PR?
Cleanup code typo in carbondata-spark module
###
CarbonDataQA1 commented on pull request #3869:
URL: https://github.com/apache/carbondata/pull/3869#issuecomment-665263897
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
VenuReddy2103 commented on a change in pull request #3857:
URL: https://github.com/apache/carbondata/pull/3857#discussion_r461385726
##
File path:
integration/hive/src/main/java/org/apache/carbondata/hive/MapredCarbonInputFormat.java
##
@@ -116,7 +117,10 @@ private static Carb
asfgit closed pull request #3860:
URL: https://github.com/apache/carbondata/pull/3860
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go t
asfgit closed pull request #3857:
URL: https://github.com/apache/carbondata/pull/3857
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go t
kevinjmh commented on a change in pull request #3867:
URL: https://github.com/apache/carbondata/pull/3867#discussion_r462058720
##
File path:
integration/spark/src/main/scala/org/apache/carbondata/view/MVRefresher.scala
##
@@ -68,19 +68,19 @@ object MVRefresher {
// Clean
CarbonDataQA1 commented on pull request #3848:
URL: https://github.com/apache/carbondata/pull/3848#issuecomment-664820504
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
CarbonDataQA1 commented on pull request #3866:
URL: https://github.com/apache/carbondata/pull/3866#issuecomment-664852730
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
akkio-97 opened a new pull request #3869:
URL: https://github.com/apache/carbondata/pull/3869
### Why is this PR needed?
Index creation for long string columns are not yet supported.
### What changes were proposed in this PR?
Exceptions are thrown if user tries to create the
ShreelekhyaG commented on a change in pull request #3774:
URL: https://github.com/apache/carbondata/pull/3774#discussion_r461343500
##
File path:
integration/spark/src/test/scala/org/apache/carbondata/geo/GeoTest.scala
##
@@ -112,6 +238,23 @@ class GeoTest extends QueryTest wi
ajantha-bhat commented on a change in pull request #3773:
URL: https://github.com/apache/carbondata/pull/3773#discussion_r461597459
##
File path:
core/src/main/java/org/apache/carbondata/core/datastore/page/encoding/FillVector.java
##
@@ -0,0 +1,345 @@
+/*
+ * Licensed to the
52 matches
Mail list logo