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

2017-03-23 Thread chenliang613
Github user chenliang613 commented on a diff in the pull request:


https://github.com/apache/incubator-carbondata/pull/672#discussion_r107839323
  
--- Diff: pom.xml ---
@@ -381,6 +389,15 @@
   
 
 
+  hive-1.2.1
--- End diff --

suggest changing to hive-1.2 for 


---
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 the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 @@
   
 
   
-  
   
+  
--- End diff --

@chenliang613 @QiangCai Qiang Cai told me that it is wrong. So I change it  
by the way


---
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 the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


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

2017-03-23 Thread chenliang613
Github user chenliang613 commented on a diff in the pull request:


https://github.com/apache/incubator-carbondata/pull/672#discussion_r107606797
  
--- Diff: dev/java-code-format-template.xml ---
@@ -34,8 +34,8 @@
   
 
   
-  
   
+  
--- End diff --

Can you explain ,why change the order ?


---
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 the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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/cenyuhai/incubator-carbondata CARBONDATA-727

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-carbondata/pull/672.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #672


commit 3a578a1b33720674d7e7adae194af11a6e7fb9de
Author: cenyuhai 
Date:   2017-03-12T15:17:40Z

add hive integration for carbon

add hive integration to assembly

alter CarbonInputFormat to implement mapred.InputFormat

add a hive serde for carbon

add hive integration to assembly

fix error in getQueryModel

add debug info

add debug info

add debug info

add debug info

fix error in CarbonRecordReader

use ArrayWritable for CarbonRecordReader

fix error in initializing CarbonRecordReader

fix error in initializing CarbonRecordReader

fix error in initializing CarbonRecordReader

fix error in initializing CarbonRecordReader

修改InputFormat的返回值

把需要查的列设置到carbon里去

fix nullpoint exception

add catalyst depedency

add catalyst depedency

add catalyst depedency

fix error in intializing carbon error

add a new hive carbon recordreader

添加把object序列化成ArrayWritable的代码

short/int等数据类型在Carbon当中实际上是Long类型

use right inspector

use right inspector

fix long can't cast int error

fix decimal cast error

column size is not equal to column type

column size is not equal to column type

column size is not equal to column type

column size is not equal to column type

fix ObjInspector error

fix ObjInspector error

fix ObjInspector error

add a new hive input split

should not combine path

add support for timestamp

clean codes

remove unused codes

support Date and TimeStamp type




---
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 the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---