Github user xuchuanyin commented on the issue:
https://github.com/apache/carbondata/pull/2379
@kumarvishal09
CARBONDATA-2620 and CARBONDATA-2618 were raised for these problems
---
Github user xuchuanyin commented on the issue:
https://github.com/apache/carbondata/pull/2379
Currently the long_string_columns does not support complex datatypes.
Carbondata will support it later in CARBONDATA-2620
---
Github user kumarvishal09 commented on the issue:
https://github.com/apache/carbondata/pull/2379
LGTM
---
Github user kumarvishal09 commented on the issue:
https://github.com/apache/carbondata/pull/2379
@xuchuanyin It is difficult for the user to configure number of rows in a
page as it depends on data, its better to handle internally based on size of
data divide the page in multiple page
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2379
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/6405/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2379
Build Success with Spark 2.2.1, Please check CI
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/5239/
---
Github user xuchuanyin commented on the issue:
https://github.com/apache/carbondata/pull/2379
retest this please
---
Github user xuchuanyin commented on the issue:
https://github.com/apache/carbondata/pull/2379
retest it please
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/2379
SDV Build Success , Please check CI
http://144.76.159.231:8080/job/ApacheSDVTests/5342/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2379
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/6396/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2379
Build Failed with Spark 2.2.1, Please check CI
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/5230/
---
Github user xuchuanyin commented on the issue:
https://github.com/apache/carbondata/pull/2379
Rebased with the latest master branch.
The second commit is to fix the review comments.
---
Github user kumarvishal09 commented on the issue:
https://github.com/apache/carbondata/pull/2379
@xuchuanyin Please remove all the changes which is not related to this pr
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2379
Build Success with Spark 2.2.1, Please check CI
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/5210/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2379
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/6372/
---
Github user xuchuanyin commented on the issue:
https://github.com/apache/carbondata/pull/2379
retest it please
---
Github user xuchuanyin commented on the issue:
https://github.com/apache/carbondata/pull/2379
retest this please
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2379
Build Failed with Spark 2.2.1, Please check CI
http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/5204/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/2379
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/6366/
---
Github user xuchuanyin commented on the issue:
https://github.com/apache/carbondata/pull/2379
@kumarvishal09
Can you review and merge it? The previous discussion can be found in #2252
---
20 matches
Mail list logo