Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/959
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/960
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/932
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/960
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/960
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/959
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/959
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/892
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/932
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/912
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/796
@PallaviSingh1992 please rebase this pr based on master
---
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
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/791
@vandana7 please resolve conflict.
---
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/carbondata/pull/941
@geetikagupta16 This issue already be solved, please close 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
GitHub user chenliang613 opened a pull request:
https://github.com/apache/carbondata/pull/953
Fix presto and hive rebase issues
1.Fix presto and hive rebase issues.
2.The test case (TestCarbonSerde.java) can't work, so remove it
temporarilyï¼ the full hive test cases
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/890
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/962
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/962
any message
---
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 commented on the issue:
https://github.com/apache/carbondata/pull/962
ok
---
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 commented on the issue:
https://github.com/apache/carbondata/pull/962
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 and wishes so
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/962
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 and wishes so
GitHub user chenliang613 opened a pull request:
https://github.com/apache/carbondata/pull/962
Only for test ci(vacuous PR-don't merge)
Only for test ci(vacuous PR-don't merge)
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/carbondata/pull/938
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 so
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/944
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/962
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 and wishes so
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/962
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 and wishes so
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/975
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/975
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/984
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 and wishes so
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/981
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/carbondata/pull/985
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/987
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/966
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/981
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/985
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/983
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/979
apache jenkins CI are not stable, let me check it at my local machine also.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/979
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/962
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/962
text
---
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 commented on the issue:
https://github.com/apache/carbondata/pull/962
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/959
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/963
@anubhav100 got the below error when verified this pr:
hive> select * from hive_table_2;
OK
Failed with exception java.io.IOException:java.lang.ClassCastExcept
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/967
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/984
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/975
@QiangCai please review this PR, thanks.
---
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/carbondata/pull/980
@mayunSaicmotor please squash all commits to one commit.
---
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 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/964#discussion_r119642951
--- Diff:
core/src/main/java/org/apache/carbondata/core/cache/dictionary/ReverseDictionaryCache.java
---
@@ -43,7 +43,7 @@
* Attribute
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/986
LGTM. verified at my local machine.
---
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/carbondata/pull/986
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/962
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/962
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/959
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 so
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/959
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1033
@rusonding your PR has problem, please rebase to the latest master
---
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
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1052
@geetikagupta16 Please verify this issue whether is valid, or not ? i
can't reproduce it at my local machine with "mvn package for
integration/presto/pom.xml"
---
If yo
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1057
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 so
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1052
@geetikagupta16 yes, now i can reproduce it, thanks.
Can you compile it based on parent pom.xml, see if can reproduce it also ?
---
If your project is set up for it, you can reply
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1060
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 so
Github user chenliang613 commented on the issue:
https://github.com/apache/incubator-carbondata/pull/839
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/816
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/881
[CARBONDATA-1020] Remove incubator info as top level project
Remove incubator info as top level project
You can merge this pull request into a Git repository by running
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/962
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/979
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/962
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/979
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
enabled
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/979
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/carbondata/pull/964#discussion_r120256136
--- Diff:
core/src/main/java/org/apache/carbondata/core/cache/dictionary/ReverseDictionaryCache.java
---
@@ -43,7 +43,7 @@
* Attribute
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1336
retest this please
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1336
retest this please
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1321
retest this please
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1358
LGTM
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1317
retest this please
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1364
@xubo245 please correct your pr title with the correct format , like :
[CARBONDATA-1483] Open auto merge when loading and inserting into
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1321
retest this please
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1321
retest this please
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1356
LGTM
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1342
LGTM
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1303
@anubhav100 please fix conflict.
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1330
@kushalsaha can you provide the detail about this PR?
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1320
retest this please
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1346
retest this please
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1345
retest this please
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1323
retest this please
---
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1286#discussion_r137939031
--- Diff:
integration/hive/src/test/java/org/apache/carbondata/hive/CarbonDictionaryDecodeReadSupportTest.java
---
@@ -0,0 +1,325
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1286#discussion_r137939004
--- Diff:
integration/hive/src/test/java/org/apache/carbondata/hive/CarbonHiveRecordReaderTest.java
---
@@ -0,0 +1,250
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1286#discussion_r137938991
--- Diff:
integration/hive/src/test/java/org/apache/carbondata/hive/CarbonHiveRecordReaderTest.java
---
@@ -0,0 +1,250
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1286
@PallaviSingh1992
1. Please remove redundant code which never be used/invoked by any other
code
2.Please enhance the "absoluteTableIdentifier and path" definition.
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1299
retest this please
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1307
LGTM
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1338
LGTM
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1286
@PallaviSingh1992 please remove TestCarbonSerde.java file
---
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1286#discussion_r138534203
--- Diff:
integration/hive/src/test/java/org/apache/carbondata/hive/CarbonHiveRecordReaderTest.java
---
@@ -0,0 +1,234
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1286#discussion_r138534300
--- Diff:
integration/hive/src/test/java/org/apache/carbondata/hive/CarbonHiveRecordReaderTest.java
---
@@ -0,0 +1,234
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1350
retest this please
---
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1347#discussion_r138603161
--- Diff: integration/hive/hive-guide.md ---
@@ -91,7 +91,6 @@ $HIVE_HOME/bin/hive
### Query data from hive table
```
-alter
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1347
LGTM
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1174
LGTM
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1282
retest this please
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/1282
retest this please
---
Github user chenliang613 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1307#discussion_r137447980
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/CarbonVectorizedRecordReader.java
---
@@ -0,0 +1,264
201 - 300 of 780 matches
Mail list logo