GitHub user Sssan520 opened a pull request:
https://github.com/apache/carbondata/pull/2436
[CARBONDATA-2682]fix create table with long_string_columns properties bugs
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/Sssan520
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2436
@jackylk can you help to review this
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2436
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2436
retest this please
---
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2436#discussion_r201542863
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/longstring/VarcharDataTypesBasicTestCase.scala
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2436#discussion_r201542881
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/longstring/VarcharDataTypesBasicTestCase.scala
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2436#discussion_r201542897
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/longstring/VarcharDataTypesBasicTestCase.scala
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2436#discussion_r201542905
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/longstring/VarcharDataTypesBasicTestCase.scala
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2436#discussion_r201542948
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
---
@@ -391,6 +391,24 @@ abstract class
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2436#discussion_r201544064
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
---
@@ -486,6 +504,60 @@ abstract class
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2436#discussion_r201544083
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
---
@@ -486,6 +504,60 @@ abstract class
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2436#discussion_r201545482
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
---
@@ -486,6 +504,60 @@ abstract class
GitHub user Sssan520 opened a pull request:
https://github.com/apache/carbondata/pull/2491
[CARBONDATA-2700][BloomDataMap] Block dropping index columns for
bloomfilter index datamap
1.Block create bloomfilter datamap index on column which its datatype is
complex type;
2.Block
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2491
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2491
retest this please
---
Github user Sssan520 closed the pull request at:
https://github.com/apache/carbondata/pull/2491
---
GitHub user Sssan520 reopened a pull request:
https://github.com/apache/carbondata/pull/2491
[CARBONDATA-2700][CARBONDATA-2730][CARBONDATA-2732][BloomDataMap] block
some oprerations of bloomfilter datamap
1.Block create bloomfilter datamap index on column which its datatype is
Github user Sssan520 closed the pull request at:
https://github.com/apache/carbondata/pull/2491
---
GitHub user Sssan520 reopened a pull request:
https://github.com/apache/carbondata/pull/2491
[CARBONDATA-2698][CARBONDATA-2700][CARBONDATA-2730][CARBONDATA-2732][BloomDataMap]
block some oprerations of bloomfilter datamap
1.Block create bloomfilter datamap index on column which
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2436#discussion_r202009599
--- Diff:
integration/spark-common/src/main/scala/org/apache/spark/sql/catalyst/CarbonDDLSqlParser.scala
---
@@ -486,6 +504,60 @@ abstract class
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2436
all code view comments has been handled.
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2436
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2436
retest this please
---
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2491#discussion_r202221482
--- Diff:
datamap/bloom/src/main/java/org/apache/carbondata/datamap/bloom/BloomCoarseGrainDataMapFactory.java
---
@@ -380,6 +380,49 @@ public void
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2491#discussion_r202221623
--- Diff:
core/src/main/java/org/apache/carbondata/core/metadata/schema/table/CarbonTable.java
---
@@ -1058,9 +1058,10 @@ public void
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2491#discussion_r202221610
--- Diff:
core/src/main/java/org/apache/carbondata/core/datamap/dev/DataMapFactory.java
---
@@ -144,4 +144,17 @@ public void validate() throws
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2491#discussion_r202221707
--- Diff:
datamap/bloom/src/main/java/org/apache/carbondata/datamap/bloom/BloomCoarseGrainDataMapFactory.java
---
@@ -380,6 +380,49 @@ public void
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2491#discussion_r202221798
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/schema/CarbonAlterTableDataTypeChangeCommand.scala
---
@@ -55,7
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2491#discussion_r202221696
--- Diff:
core/src/main/java/org/apache/carbondata/core/metadata/schema/table/CarbonTable.java
---
@@ -1071,6 +1072,10 @@ public boolean canAllow
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2491#discussion_r202221774
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/datamap/CarbonCreateDataMapCommand.scala
---
@@ -109,6 +111,19
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2491#discussion_r202226296
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/schema/CarbonAlterTableDropColumnCommand.scala
---
@@ -58,7 +58,10
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2491#discussion_r202228474
--- Diff:
integration/spark2/src/test/scala/org/apache/carbondata/datamap/bloom/BloomCoarseGrainDataMapSuite.scala
---
@@ -377,7 +382,91 @@ class
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2436
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2436
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2491
all code review comments has been handled.
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2491
retest this please
---
Github user Sssan520 closed the pull request at:
https://github.com/apache/carbondata/pull/2491
---
GitHub user Sssan520 reopened a pull request:
https://github.com/apache/carbondata/pull/2491
[CARBONDATA-2698][CARBONDATA-2700][CARBONDATA-2730][CARBONDATA-2732][BloomDataMap]
block some oprerations of bloomfilter datamap
1.Block create bloomfilter datamap index on column which
Github user Sssan520 closed the pull request at:
https://github.com/apache/carbondata/pull/2491
---
GitHub user Sssan520 reopened a pull request:
https://github.com/apache/carbondata/pull/2491
[CARBONDATA-2698][CARBONDATA-2700][CARBONDATA-2732][BloomDataMap] block
some oprerations of bloomfilter datamap
1.Block create bloomfilter datamap index on column which its datatype is
Github user Sssan520 closed the pull request at:
https://github.com/apache/carbondata/pull/2491
---
Github user Sssan520 closed the pull request at:
https://github.com/apache/carbondata/pull/2436
---
GitHub user Sssan520 opened a pull request:
https://github.com/apache/carbondata/pull/2505
[CARBONDATA-2698][CARBONDATA-2700][CARBONDATA-2732][BloomDataMap] block
some oprerations of bloomfilter datamap
1.Block create bloomfilter datamap index on column which its datatype is
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2505
There are some problems in the CI env, so I closed #2491 and opened this PR
---
GitHub user Sssan520 opened a pull request:
https://github.com/apache/carbondata/pull/2506
[CARBONDATA-2682][32K] fix create table with long_string_columns properties
bugs
Changes
This PR fixes create table with long_string_columns bugs which are:
1.create table with
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2506
There are some problems in the CI env, so I closed #2436 and opened this PR.
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2506
retest this please.
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2505
retest this please.
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2506
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2505
retest this please.
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2505
retest this please.
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2505
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2506
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2505
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2505
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2506
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2505
retest sdv please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2505
retest sdv please
---
GitHub user Sssan520 opened a pull request:
https://github.com/apache/carbondata/pull/2557
[CARBONDATA-2782]delete dead code in class 'CarbonCleanFilesCommand'
the variables(dmsãindexDms) in function processMetadata are nerver used.
Be sure to do all of the
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2557
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2557
retest this please
---
GitHub user Sssan520 opened a pull request:
https://github.com/apache/carbondata/pull/2619
[CARBONDATA-2819] Fixed cannot drop preagg datamap on table if the tabâ¦
1.Use "SET carbon.datamap.visible.{dbName}.{mainTable}.{datamapName} =
{true or false}",if datamap p
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2619
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2619
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2619
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2619
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2619
retest this please
---
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2619#discussion_r209205953
--- Diff:
examples/spark2/src/main/scala/org/apache/carbondata/examples/PreAggregateDataMapExample.scala
---
@@ -175,7 +175,7 @@ object
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2619#discussion_r209206775
--- Diff:
integration/spark2/src/main/java/org/apache/carbondata/datamap/PreAggregateDataMapProvider.java
---
@@ -77,13 +79,15 @@ private void
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2619#discussion_r209208751
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/datamap/CarbonDropDataMapCommand.scala
---
@@ -114,12 +114,6
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2619
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2619
retest this please
---
Github user Sssan520 closed the pull request at:
https://github.com/apache/carbondata/pull/2619
---
GitHub user Sssan520 opened a pull request:
https://github.com/apache/carbondata/pull/2641
[CARBONDATA-2819] Fixed cannot drop preagg datamap on table which has other
type datamaps
As we know, carbon now write preagg datamap info to main table schema and
write other type datamap
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2641
There are some problems, so I closed #2619 and opened this PR
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2641
retest this please
---
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2641#discussion_r210782671
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/datamap/CarbonDropDataMapCommand.scala
---
@@ -116,8 +116,12
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2641#discussion_r210790498
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/datamap/CarbonDropDataMapCommand.scala
---
@@ -116,8 +116,12
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2641
retest this please
---
GitHub user Sssan520 opened a pull request:
https://github.com/apache/carbondata/pull/2656
[ExternalFormat] block some operations on external format table
These operations will be blocked:
1. update data on external format table
2. delete data on external format table
3
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2656#discussion_r212604056
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/externalformat/CsvBasedCarbonTableSuite.scala
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2656#discussion_r212604083
--- Diff:
core/src/main/java/org/apache/carbondata/core/metadata/schema/table/CarbonTable.java
---
@@ -1276,4 +1278,13 @@ public static boolean
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2656#discussion_r212604024
--- Diff:
integration/spark2/src/main/scala/org/apache/spark/sql/execution/command/mutation/CarbonProjectForDeleteCommand.scala
---
@@ -49,7 +49,9
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on the issue:
https://github.com/apache/carbondata/pull/2656
retest this please
---
Github user Sssan520 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2885#discussion_r229618246
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/util/Auditor.java ---
@@ -0,0 +1,216 @@
+/*
+ * Licensed to the Apache
1 - 100 of 115 matches
Mail list logo
