GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1251
[CARBONDATA-1372]Fix link error in CarbonData documentation
The link of "building CarbonData" is error, fix it.
Be sure to do all of the following to help us incorp
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1252
[CARBONDATA-1372]Update sample.csv file name in the example of documentation
rename sample.csv file name
Be sure to do all of the following to help us incorporate your contribution
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1264
Updated hive-guide.md
Updated hive-guide.md
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/xubo245/carbondata patch-3
Alternatively you
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1361
[CARBONDATA-1481]Compaction support global sort
We should add some test cases and evaluate performance for Compaction
support global_sort.
Test cases:
1. compaction type: major
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1357#discussion_r139051979
--- Diff: docs/data-management.md ---
@@ -135,7 +135,7 @@ CLEAN FILES FOR TABLE table1
| Parameter | Default | Application | Description
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1361
@QiangCai OKï¼ it has been changed.
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1361
@QiangCai Please review it
---
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1362
[CARBONDATA-1444]Support boolean
CarbonData should support boolean data type in following aspects:
1. create table
2.insert into table values and select
3.load data
4.filter
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1364
@chenliang613 OK
---
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1364
[CARBONDATA-1483]open auto merge when loading data
auto compaction when loading data
* set DEFAULT_ENABLE_AUTO_LOAD_MERGE as true
* add test case and verify the change is ok
You can
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1361
Have SDV accident error? I haven't changed the SDV code after SDV Build
fail, but now it shows Success.
---
Github user xubo245 closed the pull request at:
https://github.com/apache/carbondata/pull/1364
---
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1405
[CARBONDATA-1547] Table should be dropped after running test
Table should be dropped after running test
You can merge this pull request into a Git repository by running:
$ git pull https
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1405
please review it @QiangCai @jackylk
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1361
please review it @jackylk
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1362
@jackylk please review it
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1362
@jackylk CI pass
---
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1406
[CARBONDATA-1506] SDV tests error in CI
Sometimes, there is error in SDV test:
== Results ==
!== Correct Answer - 1 == == Spark Answer - 1 ==
![0.0] [1.0E-5]
sometimes
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1406
Please review it @QiangCai
---
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1407
[CARBONDATA-1549] CarbonProperties should be default value after running
test
CarbonProperties should be default value after running test
if carbon properties is not default value
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1408
[CARBONDATA-1568] Optimize annotation of code
There are some improper places in code annotation by IDEA inspect code
We optimize annotation of code:
- Fix an error of Javadoc
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1404
Please review it @jackylk
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1404
@jackylk I have changed it and CI pass.
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1361
Please review it again @jackylk
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1362
Conflicts has been fixedï¼ please review it @jackylk
---
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1357
[CARBONDATA-1372]update compaction documentation
In order to easier to understand, update the description about compaction.
You can merge this pull request into a Git repository by running
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1396
[CARBONDATA-1536] Default value of carbon.bad.records.action is FORCE
Default value of carbon.bad.records.action is FORCE in source code, but the
value is fail in documentation
You can merge
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1362
Please update jar of format module @ravipesala
---
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r141994786
--- Diff:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
---
@@ -707,6 +711,10 @@
*/
public static
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r141512380
--- Diff:
core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java
---
@@ -23,6 +23,10 @@
public final class
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1362
I can run success in my computer, but I can not fix this problem in
github/jenkins. How to add "mvn clean -Pbuild-with-format install" to run my
branch in github after pushing request
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1392
I have changed indentation @jackylk
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1392
Please review and merge it. @jackylk
---
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1392
[CARBONDATA-1531] Format module should support BOOLEAN
format module support Boolean data type when I was developing Boolean
function, and there are some error in CI after push request
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1407
Other modules have not been checked at present, including spark2, core and
so on
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1361
@jackylk I has changed the title and content of this PR
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1361
@jackylk Please review it
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1362
I changed the format module.Other module can not invoke the new code in
format module.
Could you help me to deal with it? @ravipesala
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1361
@QiangCai Please review it
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1404
Please review it @QiangCai
---
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1404
[CARBONDATA-1541] There are some errors when bad_records_action is IGNORE
Fix error:
1.When table only have one column and the column data is INT, it should be
success
2.when
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1362
@jackylk please review it
---
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890810
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/filter/executer/RowLevelFilterExecuterImpl.java
---
@@ -203,7 +203,7 @@ private void
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890779
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/encoding/EncodingFactory.java
---
@@ -38,11 +38,7 @@
import
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890823
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/filter/executer/RowLevelFilterExecuterImpl.java
---
@@ -215,7 +215,7 @@ private void
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890833
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/encoding/bool/BooleanConvert.java
---
@@ -0,0 +1,63 @@
+/*
+ * Licensed
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890703
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/encoding/EncodingFactory.java
---
@@ -38,11 +38,7 @@
import
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890685
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/ColumnPage.java ---
@@ -530,6 +546,11 @@ private void putNull(int rowId
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1361#discussion_r143898742
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/datacompaction/CompactionSupportGlobalSortBigFileTest.scala
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1361#discussion_r143898738
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/datacompaction/CompactionSupportGlobalSortBigFileTest.scala
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890183
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/encoding/bool/BooleanConvert.java
---
@@ -0,0 +1,61 @@
+/*
+ * Licensed
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890228
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/ColumnPage.java ---
@@ -187,6 +179,7 @@ public static ColumnPage newPage
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890311
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/ColumnPage.java ---
@@ -271,6 +267,18 @@ private static ColumnPage
newIntPage
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890342
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/encoding/bool/BooleanEncoderMeta.java
---
@@ -0,0 +1,40 @@
+package
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890027
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/encoding/EncodingFactory.java
---
@@ -92,6 +90,10 @@ public ColumnPageDecoder
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143891060
--- Diff:
integration/spark2/src/test/scala/org/apache/carbondata/spark/testsuite/booleantype/BooleanDataTypesLoadTest.scala
---
@@ -0,0 +1,655
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1362
@jackylk I have added some descriptions for test cases in this PR.
---
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1361#discussion_r143902837
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/datacompaction/CompactionSupportGlobalSortFunctionTest.scala
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890910
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/ColumnPage.java ---
@@ -361,6 +360,10 @@ public void putData(int rowId, Object
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890886
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/store/TablePage.java
---
@@ -187,6 +188,12 @@ private void
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890846
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/encoding/bool/BooleanConvert.java
---
@@ -0,0 +1,63 @@
+/*
+ * Licensed
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1407
@jackylk Please review it.
---
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1361#discussion_r143902415
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/datacompaction/CompactionSupportGlobalSortParameterTest.scala
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890406
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/ColumnPage.java ---
@@ -321,6 +315,11 @@ private static ColumnPage
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890387
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/ColumnPage.java ---
@@ -436,6 +439,11 @@ public void putData(int rowId, Object
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890419
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/ColumnPage.java ---
@@ -485,6 +496,11 @@ private void putNull(int rowId
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1362#discussion_r143890454
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/ColumnPage.java ---
@@ -321,6 +315,11 @@ private static ColumnPage
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1404#discussion_r143897439
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/integration/spark/testsuite/dataload/LoadDataWithBadRecords.scala
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1404#discussion_r143897498
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/integration/spark/testsuite/dataload/LoadDataWithBadRecords.scala
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1430
[CARBONDATA-1444] Update documentation about Boolean data type
Add BOOLEAN data type into "supported-data-types-in-carbondata"
documentation
You can merge this pull request
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1568
[CARBONDATA-1810] Bad record path is not correct for UT
Changed bad record path in TestQueryExecutor
Be sure to do all of the following checklist to help us incorporate
your
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1523
retest this please
---
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1523#discussion_r153067752
--- Diff:
integration/spark2/src/test/scala/org/apache/carbondata/spark/testsuite/booleantype/compress/BooleanFile.scala
---
@@ -0,0 +1,59
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1568
@jackylk @QiangCai Please review it.
---
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1569
[CARBONDATA-1752] There are some scalastyle error should be optimized in
CarbonData
There are some scalastyle error should be optimized in CarbonData,
including removing useless import
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1670
retest this please
---
Github user xubo245 closed the pull request at:
https://github.com/apache/carbondata/pull/1404
---
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1670
[CARBONDATA-1899] Add CarbonData concurrency test case
Add CarbonData concurrency test case
Be sure to do all of the following checklist to help us incorporate
your contribution
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1645
retest this please
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1676
@manishgupta88 OK, I reset the unused import.
---
Github user xubo245 commented on the pull request:
https://github.com/apache/carbondata/commit/8d9babe3cb783b21791c577d1b22cc331e5ce967#commitcomment-26386268
In
core/src/test/java/org/apache/carbondata/scanner/impl/FilterScannerTest.java:
In
core/src/test/java/org/apache
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1691
[CARBONDATA-1752] Remove unused import
Remove unused imports, which were find by inspect code of IDEA Analyze
Be sure to do all of the following checklist to help us incorporate
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1670#discussion_r157973096
--- Diff:
examples/spark2/src/main/scala/org/apache/carbondata/examples/ConcurrencyTest.scala
---
@@ -0,0 +1,355 @@
+/*
+ * Licensed
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1670#discussion_r157972084
--- Diff:
examples/spark2/src/main/scala/org/apache/carbondata/examples/ConcurrencyTest.scala
---
@@ -0,0 +1,355 @@
+/*
+ * Licensed
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1676
retest this please
---
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1670#discussion_r157974643
--- Diff:
examples/spark2/src/main/scala/org/apache/carbondata/examples/ConcurrencyTest.scala
---
@@ -0,0 +1,355 @@
+/*
+ * Licensed
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1670#discussion_r157974357
--- Diff:
examples/spark2/src/main/scala/org/apache/carbondata/examples/ConcurrencyTest.scala
---
@@ -0,0 +1,355 @@
+/*
+ * Licensed
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1676
[CARBONDATA-1906] Update registerTempTable method because it was marked
deprecated
Update registerTempTable method to createOrReplaceTempView, because
registerTempTable was marked
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1670#discussion_r157698791
--- Diff:
examples/spark2/src/main/scala/org/apache/carbondata/examples/ConcurrencyTest.scala
---
@@ -0,0 +1,352 @@
+/*
+ * Licensed
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1676
retest this please
---
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1670#discussion_r157363538
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/carbondata/spark/testsuite/dataretention/ConcurrencyTest.scala
---
@@ -0,0 +1,309
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1645
@jackylk @QiangCai Please review it
---
Github user xubo245 closed the pull request at:
https://github.com/apache/carbondata/pull/1512
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/1512
@akashrn5 ok
---
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1523
[CARBONDATA-1756] Improve Boolean data compress rate by changing RLE to
SNNAPY algorithm
Improve Boolean data compress rate by changing RLE to SNNAPY algorithm
Because Boolean data
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1512
[CARBONDATA-1742] Fix NullPointerException in SegmentStatusManager
when loadFolderDetailsArray is null ,there is NullPointerException. We
should fix it.
You can merge this pull request
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1525
[CARBONDATA-1751] Make the type of exception and message correctly
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1526
[CARBONDATA-1764] Fix issue of when create table with short data type
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily
GitHub user xubo245 opened a pull request:
https://github.com/apache/carbondata/pull/1529
[CARBONDATA-1765] Remove repeat code of Boolean
Remove repeat code of Boolean
Be sure to do all of the following checklist to help us incorporate
your contribution quickly
Github user xubo245 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1515#discussion_r151832176
--- Diff:
integration/spark-common-test/src/test/scala/org/apache/spark/sql/execution/command/CarbonTableSchemaCommonSuite.scala
---
@@ -0,0 +1,69
1 - 100 of 2151 matches
Mail list logo
