Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/506
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
GitHub user chenliang613 opened a pull request:
https://github.com/apache/incubator-carbondata/pull/500
Add CI building status and Apaches license icon
Add CI building status and Apaches license icon
You can merge this pull request into a Git repository by running:
$ git pull
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/499
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/484
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/486
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/484#discussion_r94424594
--- Diff:
integration/spark-common/src/main/java/org/apache/carbondata/spark/load/DeleteLoadFolders.java
---
@@ -60,11 +61,11 @@ private
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/485
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/485#discussion_r94292038
--- Diff:
integration/spark2/src/test/scala/org/apache/spark/sql/common/util/QueryTest.scala
---
@@ -41,18 +41,11 @@ class QueryTest
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/486
add to whitelist
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/487
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/461
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 closed the pull request at:
https://github.com/apache/incubator-carbondata/pull/476
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
GitHub user chenliang613 opened a pull request:
https://github.com/apache/incubator-carbondata/pull/476
[CARBONDATA-576] Add build guide to github
Add build guide to github as per incubator mailing list comments.
You can merge this pull request into a Git repository by running
GitHub user chenliang613 opened a pull request:
https://github.com/apache/incubator-carbondata/pull/475
[CARBONDATA-575] Remove integration-testcases module
Remove integration-testcases module
You can merge this pull request into a Git repository by running:
$ git pull https
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/473#discussion_r94003843
--- Diff:
processing/src/main/java/org/apache/carbondata/processing/datatypes/GenericDataType.java
---
@@ -44,11 +44,6 @@
String
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/474#discussion_r93998269
--- Diff:
integration/spark2/src/main/scala/org/apache/carbondata/spark/thriftserver/CarbonThriftServer.scala
---
@@ -0,0 +1,64
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/471#discussion_r93937123
--- Diff:
integration/spark2/src/main/java/org/apache/carbondata/spark/vectorreader/ColumnarVectorWrapper.java
---
@@ -21,7 +21,7
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/471#discussion_r93921710
--- Diff:
integration/spark2/src/main/java/org/apache/carbondata/spark/vectorreader/ColumnarVectorWrapper.java
---
@@ -21,7 +21,7
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/469
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/449
@QiangCai please rebase this PR
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/452
Tested with DataManagementExample, ok.
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
GitHub user chenliang613 opened a pull request:
https://github.com/apache/incubator-carbondata/pull/460
[CARBONDATA-560] fix QueryExecutionException processing.
In QueryExecutionException:
Change executorService.shutdownNow
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/458
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/451
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/453
add to whitelist
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/452
please add test cases to validate the below commands after extracting:
ShowLoads
DeleteLoadById
DeleteLoadByDate
CleanFiles
---
If your project is set up
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/275
add to whitelist
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/451#discussion_r93257410
--- Diff:
core/src/main/java/org/apache/carbondata/core/cache/dictionary/DoubleArrayTrieDictionary.java
---
@@ -0,0 +1,416
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/451#discussion_r93257297
--- Diff:
core/src/main/java/org/apache/carbondata/core/cache/dictionary/DoubleArrayTrieDictionary.java
---
@@ -0,0 +1,416
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/451
@kumarvishal09 please review this PR
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/447
ok to test
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
GitHub user chenliang613 opened a pull request:
https://github.com/apache/incubator-carbondata/pull/446
[CARBONDATA-544] Delete
core/.TestFileFactory.carbondata.crc,core/Testdb.carbon
Delete test file core/.TestFileFactory.carbondata.crc,core/Testdb.carbon
You can merge this pull
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/442
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/320
add to whitelist
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/437
add to whitelist
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/434
LGTM. this fix is for branch-0.2(spark-1.x) , so can ignore travis
CI(spark-2.0).
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/421#discussion_r92093995
--- Diff:
core/src/test/java/org/apache/carbondata/scan/complextypes/PrimitiveQueryTypeTest.java
---
@@ -37,12 +40,14 @@
import
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/421
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/416
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/416#discussion_r91851006
--- Diff:
examples/spark/src/main/scala/org/apache/carbondata/examples/CarbonExample.scala
---
@@ -26,11 +28,15 @@ object CarbonExample
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/120
@ashokblend please rebase this PR
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/416#discussion_r91846766
--- Diff:
examples/spark/src/main/scala/org/apache/carbondata/examples/CarbonExample.scala
---
@@ -26,11 +28,15 @@ object CarbonExample
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/407
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/407
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/353
retest this please
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/353
ok to test
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/368
add to whitelist
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/387
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/388
@ashokblend please create dev mailing list discussion and raise Apache
JIRA ISSUE in advance.
---
If your project is set up for it, you can reply to this email and have your
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/391
é误信æ¯å¦ä¸ï¼
**Regression**
org.apache.carbondata.core.util.DataTypeUtilTest.testGetDataType
Failing for the past 1 build (Since Unstable#748 )
è¿è¡æ
GitHub user chenliang613 opened a pull request:
https://github.com/apache/incubator-carbondata/pull/392
change snapshot version number due to new structure changes as per mailing
proposal
A rich set of features are planned to be included into next release, and
more importantly
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/55
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/338
@ashokblend please correct PR title like : [CARBONDATA-]
Description of pull request.
The below checklist is for your reference,don't need copy it as description
content
GitHub user chenliang613 opened a pull request:
https://github.com/apache/incubator-carbondata/pull/334
update pom version
In branch-0.2, the next version number should be 0.2.1
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/329
LGTM
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
201 - 255 of 255 matches
Mail list logo