Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2334
Build Success with Spark 2.2.1, Please check CI
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/5358/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2334
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/6526/
---
Github user sv71294 commented on the issue:
https://github.com/apache/carbondata/pull/2334
@chenliang613 please check this image, condition logic was missing
Github user sv71294 commented on the issue:
https://github.com/apache/carbondata/pull/2334
@chenliang613 issue is not about the result, in current master filter is
getting handled by presto as it is not generated on connector layer, now with
this PR it will be generated at connector
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/2334
@sv71294
i can also run successfully at current master with the below test case, so
not sure this test case can cover this issue which be fixed in the PR.
```
test("test the
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/2334
SDV Build Success , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/5392/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2334
Build Success with Spark 2.2.1, Please check CI
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/5303/
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/2334
SDV Build Success , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/5391/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2334
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/6472/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2334
Build Success with Spark 2.2.1, Please check CI
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/5300/
---
Github user sv71294 commented on the issue:
https://github.com/apache/carbondata/pull/2334
Retest this please
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2334
Build Failed with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/6469/
---
Github user sv71294 commented on the issue:
https://github.com/apache/carbondata/pull/2334
@chenliang613 I have added the required test case for greater than
expresion, please check it now
---
Github user sv71294 commented on the issue:
https://github.com/apache/carbondata/pull/2334
@chenliang613 adding the test case to cover timestamp greater than
expression issue
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/2334
For "[CARBONDATA-2515] OR filter Expression issue" , verified.
```
presto:default> select * from carbon_table where ID='No.11' and
country='country0' and population > 100 or
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/2334
@sv71294 thank you for your contribution.
In this pr, i didn't find the test case to cover "Timestamp greaterthan
expression"
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2334
Build Success with Spark 2.2.1, Please check CI
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/5286/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2334
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/6455/
---
Github user sv71294 commented on the issue:
https://github.com/apache/carbondata/pull/2334
retest this please
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2334
Build Failed with Spark 2.2.1, Please check CI
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/5282/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2334
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/6451/
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/2334
retest this please
---
Github user chenliang613 commented on the issue:
https://github.com/apache/carbondata/pull/2334
add to whitelist
---
Github user sv71294 commented on the issue:
https://github.com/apache/carbondata/pull/2334
retest this please
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/2334
retest this please
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2334
Build Failed with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/6146/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2334
Build Success with Spark 2.2.1, Please check CI
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/4983/
---
Github user jackylk commented on the issue:
https://github.com/apache/carbondata/pull/2334
retest this please
---
Github user jackylk commented on the issue:
https://github.com/apache/carbondata/pull/2334
LGTM
---
Github user sv71294 commented on the issue:
https://github.com/apache/carbondata/pull/2334
Please review it now
---
Github user xubo245 commented on the issue:
https://github.com/apache/carbondata/pull/2334
Please optimize the code style of this PR
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2334
Can one of the admins verify this patch?
---
32 matches
Mail list logo