Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/573
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 CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/573
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/797/
---
If your project is set up for it, you can reply to this email and
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/573
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/796/
---
If your project is set up for it, you can reply to this email and
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/573
Build Success with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/795/
---
If your project is set up for it, you can reply to this email and
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/573
Build Failed with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/794/
---
If your project is set up for it, you can reply to this email and
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/573
@QiangCai please check the added data.csv in this PR, I remember there is a
common data.csv for all test cases.
---
If your project is set up for it, you can reply to this email and
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/573
ok, goods fine to me. please rebase the code
---
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
Github user JihongMA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/573
CSVReader is only for data loading, processing module is more appropriate
component to hold it. to me, Carbon Hadoop component is primarily for Carbon's
input/output format for hadoop
Github user jackylk commented on the issue:
https://github.com/apache/incubator-carbondata/pull/573
@JihongMA Thanks for working on this, can you explain more about this PR,
why CSVReader is moved to processing module?
---
If your project is set up for it, you can reply to this
Github user CarbonDataQA commented on the issue:
https://github.com/apache/incubator-carbondata/pull/573
Build Failed with Spark 1.6.2, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/754/
---
If your project is set up for it, you can reply to this email and
10 matches
Mail list logo