[GitHub] [carbondata] xubo245 commented on a change in pull request #3273: [CARBONDATA-3342] Fix the IllegalArgumentException when using filter and result is null.

2020-04-29 Thread GitBox
xubo245 commented on a change in pull request #3273: URL: https://github.com/apache/carbondata/pull/3273#discussion_r417794487 ## File path: sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java ## @@ -87,9 +91,13 @@ public boolean hasNext() throws IOException

[GitHub] [carbondata] jackylk commented on pull request #3301: [CARBONDATA-3446] Support read schema of complex data type from carbon file folder path

2020-04-29 Thread GitBox
jackylk commented on pull request #3301: URL: https://github.com/apache/carbondata/pull/3301#issuecomment-621650164 LGTM This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [carbondata] jackylk commented on a change in pull request #3273: [CARBONDATA-3342] Fix the IllegalArgumentException when using filter and result is null.

2020-04-29 Thread GitBox
jackylk commented on a change in pull request #3273: URL: https://github.com/apache/carbondata/pull/3273#discussion_r417793359 ## File path: sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java ## @@ -87,9 +91,13 @@ public boolean hasNext() throws IOException

[GitHub] [carbondata] jackylk commented on pull request #3317: [CARBONDATA-3461] Carbon SDK support filter equal values set.

2020-04-29 Thread GitBox
jackylk commented on pull request #3317: URL: https://github.com/apache/carbondata/pull/3317#issuecomment-621648644 retest this please This is an automated message from the Apache Git Service. To respond to the message, pleas

[jira] [Created] (CARBONDATA-3788) Fix insert failure during global sort with huge data in new insert flow

2020-04-29 Thread Ajantha Bhat (Jira)
Ajantha Bhat created CARBONDATA-3788: Summary: Fix insert failure during global sort with huge data in new insert flow Key: CARBONDATA-3788 URL: https://issues.apache.org/jira/browse/CARBONDATA-3788

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3732: [WIP] Fix insert failure during global sort with huge data in new insert flow

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3732: URL: https://github.com/apache/carbondata/pull/3732#issuecomment-621637038 Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/1182/

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3732: [WIP] Fix insert failure during global sort with huge data in new insert flow

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3732: URL: https://github.com/apache/carbondata/pull/3732#issuecomment-621635299 Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/2898/ -

[jira] [Resolved] (CARBONDATA-3787) MemoryLeak in data load and compaction

2020-04-29 Thread Ajantha Bhat (Jira)
[ https://issues.apache.org/jira/browse/CARBONDATA-3787?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Ajantha Bhat resolved CARBONDATA-3787. -- Fix Version/s: 2.0.0 Resolution: Fixed > MemoryLeak in data load and compac

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3301: [CARBONDATA-3446] Support read schema of complex data type from carbon file folder path

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3301: URL: https://github.com/apache/carbondata/pull/3301#issuecomment-621632720 Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/2897/ -

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3301: [CARBONDATA-3446] Support read schema of complex data type from carbon file folder path

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3301: URL: https://github.com/apache/carbondata/pull/3301#issuecomment-621632518 Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/1181/

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3273: [CARBONDATA-3342] Fix the IllegalArgumentException when using filter and result is null.

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3273: URL: https://github.com/apache/carbondata/pull/3273#issuecomment-621632527 Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/1183/

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3273: [CARBONDATA-3342] Fix the IllegalArgumentException when using filter and result is null.

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3273: URL: https://github.com/apache/carbondata/pull/3273#issuecomment-621631932 Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/2899/ -

[GitHub] [carbondata] ajantha-bhat commented on pull request #3706: [CARBONDATA-3787]Fixed MemoryLeak in data load and compaction

2020-04-29 Thread GitBox
ajantha-bhat commented on pull request #3706: URL: https://github.com/apache/carbondata/pull/3706#issuecomment-621629210 LGTM This is an automated message from the Apache Git Service. To respond to the message, please log on

[jira] [Created] (CARBONDATA-3787) MemoryLeak in data load and compaction

2020-04-29 Thread Ajantha Bhat (Jira)
Ajantha Bhat created CARBONDATA-3787: Summary: MemoryLeak in data load and compaction Key: CARBONDATA-3787 URL: https://issues.apache.org/jira/browse/CARBONDATA-3787 Project: CarbonData

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3706: [WIP]Fixed MemoryLeak in data load and compaction

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3706: URL: https://github.com/apache/carbondata/pull/3706#issuecomment-621610204 Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/2895/ -

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3706: [WIP]Fixed MemoryLeak in data load and compaction

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3706: URL: https://github.com/apache/carbondata/pull/3706#issuecomment-621609299 Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/1179/

[GitHub] [carbondata] xubo245 commented on a change in pull request #3273: [CARBONDATA-3342] Fix the IllegalArgumentException when using filter and result is null.

2020-04-29 Thread GitBox
xubo245 commented on a change in pull request #3273: URL: https://github.com/apache/carbondata/pull/3273#discussion_r417740595 ## File path: sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java ## @@ -87,9 +91,13 @@ public boolean hasNext() throws IOException

[GitHub] [carbondata] xubo245 commented on a change in pull request #3273: [CARBONDATA-3342] Fix the IllegalArgumentException when using filter and result is null.

2020-04-29 Thread GitBox
xubo245 commented on a change in pull request #3273: URL: https://github.com/apache/carbondata/pull/3273#discussion_r417740595 ## File path: sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java ## @@ -87,9 +91,13 @@ public boolean hasNext() throws IOException

[GitHub] [carbondata] ajantha-bhat opened a new pull request #3732: [WIP] Fix insert failure during global sort with huge data in new insert flow

2020-04-29 Thread GitBox
ajantha-bhat opened a new pull request #3732: URL: https://github.com/apache/carbondata/pull/3732 ### Why is this PR needed? ### What changes were proposed in this PR? ### Does this PR introduce any user interface change? - No - Yes. (please explain the

[GitHub] [carbondata] xubo245 commented on a change in pull request #3273: [CARBONDATA-3342] Fix the IllegalArgumentException when using filter and result is null.

2020-04-29 Thread GitBox
xubo245 commented on a change in pull request #3273: URL: https://github.com/apache/carbondata/pull/3273#discussion_r417739818 ## File path: sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java ## @@ -87,9 +91,13 @@ public boolean hasNext() throws IOException

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3301: [CARBONDATA-3446] Support read schema of complex data type from carbon file folder path

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3301: URL: https://github.com/apache/carbondata/pull/3301#issuecomment-621591410 Build Failed with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/2896/ -

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3301: [CARBONDATA-3446] Support read schema of complex data type from carbon file folder path

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3301: URL: https://github.com/apache/carbondata/pull/3301#issuecomment-621591143 Build Failed with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/1180/

[GitHub] [carbondata] xubo245 commented on pull request #3301: [CARBONDATA-3446] Support read schema of complex data type from carbon file folder path

2020-04-29 Thread GitBox
xubo245 commented on pull request #3301: URL: https://github.com/apache/carbondata/pull/3301#issuecomment-621588986 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [carbondata] xubo245 commented on a change in pull request #3301: [CARBONDATA-3446] Support read schema of complex data type from carbon file folder path

2020-04-29 Thread GitBox
xubo245 commented on a change in pull request #3301: URL: https://github.com/apache/carbondata/pull/3301#discussion_r417728947 ## File path: sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonSchemaReader.java ## @@ -283,6 +280,96 @@ public static Schema readSchemaInIn

[GitHub] [carbondata] xubo245 commented on a change in pull request #3301: [CARBONDATA-3446] Support read schema of complex data type from carbon file folder path

2020-04-29 Thread GitBox
xubo245 commented on a change in pull request #3301: URL: https://github.com/apache/carbondata/pull/3301#discussion_r417729040 ## File path: sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonSchemaReader.java ## @@ -283,6 +280,96 @@ public static Schema readSchemaInIn

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3731: [CARBONDATA-3786] presto carbon reader should use tablePath from hive catalog

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3731: URL: https://github.com/apache/carbondata/pull/3731#issuecomment-621586745 Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/2894/ -

[GitHub] [carbondata] xubo245 commented on a change in pull request #3301: [CARBONDATA-3446] Support read schema of complex data type from carbon file folder path

2020-04-29 Thread GitBox
xubo245 commented on a change in pull request #3301: URL: https://github.com/apache/carbondata/pull/3301#discussion_r417728835 ## File path: sdk/sdk/src/test/java/org/apache/carbondata/sdk/file/CSVCarbonWriterTest.java ## @@ -676,4 +680,194 @@ public void testWithTableProperti

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3731: [CARBONDATA-3786] presto carbon reader should use tablePath from hive catalog

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3731: URL: https://github.com/apache/carbondata/pull/3731#issuecomment-621585703 Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/1178/

[GitHub] [carbondata] xubo245 commented on a change in pull request #3301: [CARBONDATA-3446] Support read schema of complex data type from carbon file folder path

2020-04-29 Thread GitBox
xubo245 commented on a change in pull request #3301: URL: https://github.com/apache/carbondata/pull/3301#discussion_r417728677 ## File path: sdk/sdk/src/test/java/org/apache/carbondata/sdk/file/CSVCarbonWriterTest.java ## @@ -676,4 +680,194 @@ public void testWithTableProperti

[GitHub] [carbondata] xubo245 commented on a change in pull request #3301: [CARBONDATA-3446] Support read schema of complex data type from carbon file folder path

2020-04-29 Thread GitBox
xubo245 commented on a change in pull request #3301: URL: https://github.com/apache/carbondata/pull/3301#discussion_r417728182 ## File path: sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonSchemaReader.java ## @@ -283,6 +280,96 @@ public static Schema readSchemaInIn

[GitHub] [carbondata] xubo245 commented on a change in pull request #3301: [CARBONDATA-3446] Support read schema of complex data type from carbon file folder path

2020-04-29 Thread GitBox
xubo245 commented on a change in pull request #3301: URL: https://github.com/apache/carbondata/pull/3301#discussion_r417728009 ## File path: sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonSchemaReader.java ## @@ -283,6 +280,96 @@ public static Schema readSchemaInIn

[GitHub] [carbondata] xubo245 commented on a change in pull request #3301: [CARBONDATA-3446] Support read schema of complex data type from carbon file folder path

2020-04-29 Thread GitBox
xubo245 commented on a change in pull request #3301: URL: https://github.com/apache/carbondata/pull/3301#discussion_r417727843 ## File path: sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonSchemaReader.java ## @@ -283,6 +280,96 @@ public static Schema readSchemaInIn

[GitHub] [carbondata] xubo245 commented on a change in pull request #3301: [CARBONDATA-3446] Support read schema of complex data type from carbon file folder path

2020-04-29 Thread GitBox
xubo245 commented on a change in pull request #3301: URL: https://github.com/apache/carbondata/pull/3301#discussion_r417726855 ## File path: core/src/main/java/org/apache/carbondata/core/metadata/datatype/ArrayType.java ## @@ -65,4 +73,11 @@ public DataType getElementType() {

[GitHub] [carbondata] jackylk commented on a change in pull request #3273: [CARBONDATA-3342] Fix the IllegalArgumentException when using filter and result is null.

2020-04-29 Thread GitBox
jackylk commented on a change in pull request #3273: URL: https://github.com/apache/carbondata/pull/3273#discussion_r417718445 ## File path: sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java ## @@ -87,9 +91,13 @@ public boolean hasNext() throws IOException

[GitHub] [carbondata] jackylk commented on a change in pull request #3273: [CARBONDATA-3342] Fix the IllegalArgumentException when using filter and result is null.

2020-04-29 Thread GitBox
jackylk commented on a change in pull request #3273: URL: https://github.com/apache/carbondata/pull/3273#discussion_r417718090 ## File path: sdk/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonReader.java ## @@ -87,9 +91,13 @@ public boolean hasNext() throws IOException

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3731: [CARBONDATA-3786] presto carbon reader should use tablePath from hive catalog

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3731: URL: https://github.com/apache/carbondata/pull/3731#issuecomment-621385076 Build Failed with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/1177/

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3731: [CARBONDATA-3786] presto carbon reader should use tablePath from hive catalog

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3731: URL: https://github.com/apache/carbondata/pull/3731#issuecomment-621381896 Build Failed with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/2893/ -

[GitHub] [carbondata] ajantha-bhat commented on pull request #3731: [CARBONDATA-3786] presto carbon reader should use tablePath from hive catalog

2020-04-29 Thread GitBox
ajantha-bhat commented on pull request #3731: URL: https://github.com/apache/carbondata/pull/3731#issuecomment-621322822 @QiangCai , @kunal642 : please check. handled for both prestodb and prestosql This is an automated me

[jira] [Created] (CARBONDATA-3786) presto carbon reader should use tablePath from hive catalog

2020-04-29 Thread Ajantha Bhat (Jira)
Ajantha Bhat created CARBONDATA-3786: Summary: presto carbon reader should use tablePath from hive catalog Key: CARBONDATA-3786 URL: https://issues.apache.org/jira/browse/CARBONDATA-3786 Project:

[GitHub] [carbondata] ajantha-bhat opened a new pull request #3731: [WIP] presto carbon reader should use tablePath from hive catalog

2020-04-29 Thread GitBox
ajantha-bhat opened a new pull request #3731: URL: https://github.com/apache/carbondata/pull/3731 ### Why is this PR needed? In upgrade scenarios of 1.6 to 2.0, when sparl.sql.warehouse is not configured. Hive storage location is not proper. so, presto carbon integration should use

[GitHub] [carbondata] dependabot[bot] commented on pull request #3456: Bump solr.version from 6.3.0 to 8.3.0 in /datamap/lucene

2020-04-29 Thread GitBox
dependabot[bot] commented on pull request #3456: URL: https://github.com/apache/carbondata/pull/3456#issuecomment-621162047 Dependabot tried to update this pull request, but something went wrong. We're looking into it, but in the meantime you can retry the update by commenting `@dependabot

[GitHub] [carbondata] dependabot[bot] commented on pull request #3447: Bump dep.jackson.version from 2.6.5 to 2.10.1 in /store/sdk

2020-04-29 Thread GitBox
dependabot[bot] commented on pull request #3447: URL: https://github.com/apache/carbondata/pull/3447#issuecomment-621162012 Dependabot tried to update this pull request, but something went wrong. We're looking into it, but in the meantime you can retry the update by commenting `@dependabot

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3729: [WIP] Allow force refresh table If table already exist in hive

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3729: URL: https://github.com/apache/carbondata/pull/3729#issuecomment-621158828 Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/1176/

[GitHub] [carbondata] jackylk commented on pull request #3700: [CARBONDATA-3784] Added spark binary version to related modules

2020-04-29 Thread GitBox
jackylk commented on pull request #3700: URL: https://github.com/apache/carbondata/pull/3700#issuecomment-621156315 LGTM This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3729: [WIP] Allow force refresh table If table already exist in hive

2020-04-29 Thread GitBox
CarbonDataQA1 commented on pull request #3729: URL: https://github.com/apache/carbondata/pull/3729#issuecomment-621152864 Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/2892/ -

[jira] [Resolved] (CARBONDATA-3778) Change the way of mv module get database location

2020-04-29 Thread Akash R Nilugal (Jira)
[ https://issues.apache.org/jira/browse/CARBONDATA-3778?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Akash R Nilugal resolved CARBONDATA-3778. - Fix Version/s: 2.0.0 Resolution: Fixed > Change the way of mv module

[GitHub] [carbondata] akashrn5 commented on pull request #3727: [CARBONDATA-3778] Change conf.getProperty("carbon.storelocation") to CarbonEvn.getDatabaseLocation in mv module

2020-04-29 Thread GitBox
akashrn5 commented on pull request #3727: URL: https://github.com/apache/carbondata/pull/3727#issuecomment-621138371 LGTM This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[jira] [Updated] (CARBONDATA-3773) Skip Validate partition info in Indexserver count star flow

2020-04-29 Thread Akash R Nilugal (Jira)
[ https://issues.apache.org/jira/browse/CARBONDATA-3773?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Akash R Nilugal updated CARBONDATA-3773: Fix Version/s: 2.0.0 > Skip Validate partition info in Indexserver count star

[jira] [Resolved] (CARBONDATA-3773) Skip Validate partition info in Indexserver count star flow

2020-04-29 Thread Akash R Nilugal (Jira)
[ https://issues.apache.org/jira/browse/CARBONDATA-3773?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Akash R Nilugal resolved CARBONDATA-3773. - Resolution: Fixed > Skip Validate partition info in Indexserver count star f