[GitHub] incubator-carbondata pull request #315: [Carbondata-388] Remove useless Carb...

2016-11-17 Thread cenyuhai
Github user cenyuhai closed the pull request at: https://github.com/apache/incubator-carbondata/pull/315 --- 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, or if

[GitHub] incubator-carbondata issue #328: [CARBONDATA-374] Support smallint

2016-11-20 Thread cenyuhai
Github user cenyuhai commented on the issue: https://github.com/apache/incubator-carbondata/pull/328 @Hexiaoqiao There is a doc about what datatype carbon are supported. https://cwiki.apache.org/confluence/display/CARBONDATA/Carbon+Data+Types Short is not working because

[GitHub] incubator-carbondata pull request #328: [CARBONDATA-374] Support smallint

2016-12-03 Thread cenyuhai
Github user cenyuhai closed the pull request at: https://github.com/apache/incubator-carbondata/pull/328 --- 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, or if

[GitHub] incubator-carbondata pull request #391: [CARBONDATA-374] support smallint

2016-12-04 Thread cenyuhai
GitHub user cenyuhai opened a pull request: https://github.com/apache/incubator-carbondata/pull/391 [CARBONDATA-374] support smallint **What changes were proposed in this pull request?** support smallint type **How to test?** Test with TestCreateTable.scala

[GitHub] incubator-carbondata issue #391: [CARBONDATA-374] support smallint

2016-12-04 Thread cenyuhai
Github user cenyuhai commented on the issue: https://github.com/apache/incubator-carbondata/pull/391 @jackylk @lion-x pls review the codes --- 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] incubator-carbondata issue #391: [CARBONDATA-374] support smallint

2016-12-05 Thread cenyuhai
Github user cenyuhai commented on the issue: https://github.com/apache/incubator-carbondata/pull/391 @ravipesala @chenliang613 --- 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

[GitHub] incubator-carbondata pull request #672: [CARBONDATA-727][WIP] add hive integ...

2017-03-19 Thread cenyuhai
GitHub user cenyuhai opened a pull request: https://github.com/apache/incubator-carbondata/pull/672 [CARBONDATA-727][WIP] add hive integration for carbon add support for carbondata You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-carbondata pull request #672: [CARBONDATA-727][WIP] add hive integ...

2017-03-23 Thread cenyuhai
Github user cenyuhai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/672#discussion_r107607409 --- Diff: dev/java-code-format-template.xml --- @@ -34,8 +34,8

[GitHub] incubator-carbondata pull request #672: [CARBONDATA-815] add hive integratio...

2017-03-24 Thread cenyuhai
Github user cenyuhai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/672#discussion_r108026227 --- Diff: integration/hive/src/main/java/org/apache/carbondata/hive/MapredCarbonOutputFormat.java --- @@ -0,0 +1,49

[GitHub] incubator-carbondata pull request #672: [CARBONDATA-815] add hive integratio...

2017-03-25 Thread cenyuhai
Github user cenyuhai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/672#discussion_r108030455 --- Diff: integration/hive/src/main/java/org/apache/carbondata/hive/MapredCarbonInputFormat.java --- @@ -0,0 +1,99

[GitHub] incubator-carbondata pull request #672: [CARBONDATA-815] add hive integratio...

2017-03-25 Thread cenyuhai
Github user cenyuhai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/672#discussion_r108030714 --- Diff: integration/hive/src/main/java/org/apache/carbondata/hive/MapredCarbonOutputFormat.java --- @@ -0,0 +1,49

[GitHub] incubator-carbondata pull request #672: [CARBONDATA-815] add hive integratio...

2017-04-05 Thread cenyuhai
Github user cenyuhai closed the pull request at: https://github.com/apache/incubator-carbondata/pull/672 --- 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, or if

[GitHub] incubator-carbondata pull request #740: add hive integration for carbon

2017-04-05 Thread cenyuhai
GitHub user cenyuhai opened a pull request: https://github.com/apache/incubator-carbondata/pull/740 add hive integration for carbon add basic hive integration for carbon You can merge this pull request into a Git repository by running: $ git pull https://github.com/cenyuhai

[GitHub] incubator-carbondata issue #740: add hive integration for carbon

2017-04-05 Thread cenyuhai
Github user cenyuhai commented on the issue: https://github.com/apache/incubator-carbondata/pull/740 @QiangCai @chenliang613 --- 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

[GitHub] incubator-carbondata pull request #740: [CARBONDATA-815] add hive integratio...

2017-04-05 Thread cenyuhai
Github user cenyuhai closed the pull request at: https://github.com/apache/incubator-carbondata/pull/740 --- 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, or if

[GitHub] incubator-carbondata issue #751: [CARBONDATA-816] Added Example for Hive Int...

2017-04-10 Thread cenyuhai
Github user cenyuhai commented on the issue: https://github.com/apache/incubator-carbondata/pull/751 @anubhav100 can you create a new project under examples and move these codes? examples --flink --spark --spark2 --hive -your codes

[GitHub] incubator-carbondata pull request #751: [CARBONDATA-816] Added Example for H...

2017-04-10 Thread cenyuhai
Github user cenyuhai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/751#discussion_r110684928 --- Diff: integration/hive/pom.xml --- @@ -64,6 +64,79 @@ compile

[GitHub] incubator-carbondata pull request #751: [CARBONDATA-816] Added Example for H...

2017-04-10 Thread cenyuhai
Github user cenyuhai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/751#discussion_r110718139 --- Diff: integration/hive/pom.xml --- @@ -64,6 +64,79 @@ compile

[GitHub] incubator-carbondata pull request #751: [CARBONDATA-816] Added Example for H...

2017-04-10 Thread cenyuhai
Github user cenyuhai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/751#discussion_r110718342 --- Diff: integration/hive/pom.xml --- @@ -64,6 +64,79 @@ compile

[GitHub] incubator-carbondata pull request #751: [CARBONDATA-816] Added Example for H...

2017-04-10 Thread cenyuhai
Github user cenyuhai commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/751#discussion_r110718467 --- Diff: integration/hive/pom.xml --- @@ -64,6 +64,79 @@ compile

[GitHub] incubator-carbondata issue #751: [CARBONDATA-816] Added Example for Hive Int...

2017-04-10 Thread cenyuhai
Github user cenyuhai commented on the issue: https://github.com/apache/incubator-carbondata/pull/751 I test this example in my computer, it's very good. But I think you should remove some unnecessary dependencies. --- If your project is set up for it, you can reply to this