Github user jackylk commented on the issue:
https://github.com/apache/carbondata/pull/3004
LGTM
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3004
Build Success with Spark 2.2.1, Please check CI
http://95.216.28.178:8080/job/ApacheCarbonPRBuilder1/2234/
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3004
Build Success with Spark 2.3.2, Please check CI
http://136.243.101.176:8080/job/carbondataprbuilder2.3/10213/
---
Github user SteNicholas commented on the issue:
https://github.com/apache/carbondata/pull/3004
@jackylk OK.I'd like to refactory on carbondata-hive module based on
@ravipesala update.
---
Github user CarbonDataQA commented on the issue:
https://github.com/apache/carbondata/pull/3004
Build Success with Spark 2.1.0, Please check CI
http://136.243.101.176:8080/job/ApacheCarbonPRBuilder2.1/1961/
---
Github user jackylk commented on the issue:
https://github.com/apache/carbondata/pull/3004
@SteNicholas Let's first merge this PR and then you can start refactory on
carbondata-hive module
---
Github user SteNicholas commented on the issue:
https://github.com/apache/carbondata/pull/3004
@ravipesala Could I refactor the current Hive integration code with you?
---
Github user ravipesala commented on the issue:
https://github.com/apache/carbondata/pull/3004
@SteNicholas This PR does not work directly on the Hive integration as we
need to set the right serde, inputformat and output format. We are planning to
refactor the current Hive integration
Github user SteNicholas commented on the issue:
https://github.com/apache/carbondata/pull/3004
@ravipesala This pull request doesn't work in #3012 .HiveExample run
exception through your changes.But this update does work in beeline.Therefore
will you check the cause of HiveExample