[GitHub] incubator-carbondata pull request #329: [CARBONDATA-388] Remove useless file...

2016-11-17 Thread cenyuhai
GitHub user cenyuhai opened a pull request:

https://github.com/apache/incubator-carbondata/pull/329

[CARBONDATA-388] Remove useless file CarbonFileFolderComparator.java

Remove useless file CarbonFileFolderComparator.java

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cenyuhai/incubator-carbondata CARBONDATA-388

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

https://github.com/apache/incubator-carbondata/pull/329.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 #329


commit a2286ba1037e9a677712e13e2e92365844e97a88
Author: cenyuhai <261810...@qq.com>
Date:   2016-11-17T16:16:26Z

remove useless file CarbonFileFolderComparator.java




---
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 #328: [CARBONDATA-374] Support smallint

2016-11-17 Thread cenyuhai
GitHub user cenyuhai opened a pull request:

https://github.com/apache/incubator-carbondata/pull/328

[CARBONDATA-374] Support smallint

Why raise this pull request?
Support smallint.

How to test?
Test with TestLoadDataWithHiveSyntax

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cenyuhai/incubator-carbondata CARBONDATA-374

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

https://github.com/apache/incubator-carbondata/pull/328.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 #328


commit cde380c6aa1f2338ec71a22f857aef0cd011ce41
Author: cenyuhai <261810...@qq.com>
Date:   2016-11-17T15:49:41Z

support smallint




---
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 #293: [CARBONDATA-374] Support smallint ty...

2016-11-17 Thread cenyuhai
Github user cenyuhai closed the pull request at:

https://github.com/apache/incubator-carbondata/pull/293


---
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 #315: [Carbondata-388]

2016-11-13 Thread cenyuhai
GitHub user cenyuhai opened a pull request:

https://github.com/apache/incubator-carbondata/pull/315

[Carbondata-388] 

Remove useless CarbonFileFolderComparator.java

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cenyuhai/incubator-carbondata CARBONDATA-388

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

https://github.com/apache/incubator-carbondata/pull/315.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 #315


commit 90089ddbe4b17f2b74b9c1e4ca0a8c49c4754b6a
Author: cenyuhai <261810...@qq.com>
Date:   2016-11-03T13:52:47Z

Change thrift files' name to a unified form

commit dcada1cccb474d694bd207a460c3426340c3187c
Author: cenyuhai <261810...@qq.com>
Date:   2016-11-03T13:55:23Z

Merge remote-tracking branch 'remotes/cenyuhai/master'

commit a8841d3188e7174d56d8c589f5fe4b2fa0189dcc
Author: 岑玉海 <261810...@qq.com>
Date:   2016-11-10T15:28:07Z

Merge pull request #1 from apache/master

Merge latest code

commit 448c1deee20ab3813fc304eb4b61d06c8fd54389
Author: cenyuhai <261810...@qq.com>
Date:   2016-11-10T15:39:25Z

Merge branch 'master' of https://github.com/cenyuhai/incubator-carbondata

commit 6360835379aa664eb52a2f6caef0aae2a4c86c54
Author: cenyuhai <261810...@qq.com>
Date:   2016-11-14T06:22:09Z

Remove Useless File CarbonFileFolderComparator.java




---
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.
---


Re: List the supported datatypes in carbondata

2016-11-04 Thread cenyuhai
I think we should make it clear that what datatypes are supported in
carbondata.

these types are confused (int or integer, short or smallint, long or bigint,
double or numeric)

some datatypes are not supported now, (numeric, integer, short, smallint,
long, numeric).

We should tell users what datatypes are supported in docs which is not
existed now.

We need a doc like
https://cwiki.apache.org/confluence/display/Hive/LanguageManual+Types

I suggest that we support (string, int, smallint, bigint, float, double,
decimal, timestamp, array, struct) at first.




--
View this message in context: 
http://apache-carbondata-mailing-list-archive.1130556.n5.nabble.com/List-the-supported-datatypes-in-carbondata-tp2419p2659.html
Sent from the Apache CarbonData Mailing List archive mailing list archive at 
Nabble.com.


[GitHub] incubator-carbondata pull request #293: [CARBONDATA-374] Support smallint ty...

2016-11-03 Thread cenyuhai
Github user cenyuhai commented on a diff in the pull request:

https://github.com/apache/incubator-carbondata/pull/293#discussion_r86494087
  
--- Diff: 
integration/spark/src/test/scala/org/apache/spark/sql/TestCreateTable.scala ---
@@ -0,0 +1,30 @@
+/*
--- End diff --

ok


---
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 #293: [Carbondata-374] Support smallint ty...

2016-11-03 Thread cenyuhai
GitHub user cenyuhai opened a pull request:

https://github.com/apache/incubator-carbondata/pull/293

[Carbondata-374] Support smallint type

What changes were proposed in this pull request?

support smallint type


How to test?
Test with TestCreateTable.scala

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cenyuhai/incubator-carbondata CARBONDATA-374

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

https://github.com/apache/incubator-carbondata/pull/293.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 #293


commit 90089ddbe4b17f2b74b9c1e4ca0a8c49c4754b6a
Author: cenyuhai <261810...@qq.com>
Date:   2016-11-03T13:52:47Z

Change thrift files' name to a unified form

commit dcada1cccb474d694bd207a460c3426340c3187c
Author: cenyuhai <261810...@qq.com>
Date:   2016-11-03T13:55:23Z

Merge remote-tracking branch 'remotes/cenyuhai/master'

commit 60ae86a3d51ae7b1988f43952e389d3d9cb9d39c
Author: cenyuhai <261810...@qq.com>
Date:   2016-11-03T18:11:32Z

support smallint 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.
---


[GitHub] incubator-carbondata pull request #291: Carbondata 351

2016-11-03 Thread cenyuhai
GitHub user cenyuhai opened a pull request:

https://github.com/apache/incubator-carbondata/pull/291

Carbondata 351

**Change thrift files' name to a unified form**
rename carbondataindex.thrift  to carbondata_index.thrift
rename carbondata_meta.thrift to carbondata_metadata.thrift

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cenyuhai/incubator-carbondata CARBONDATA-351

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

https://github.com/apache/incubator-carbondata/pull/291.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 #291


commit 90089ddbe4b17f2b74b9c1e4ca0a8c49c4754b6a
Author: cenyuhai <261810...@qq.com>
Date:   2016-11-03T13:52:47Z

Change thrift files' name to a unified form

commit dcada1cccb474d694bd207a460c3426340c3187c
Author: cenyuhai <261810...@qq.com>
Date:   2016-11-03T13:55:23Z

Merge remote-tracking branch 'remotes/cenyuhai/master'




---
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.
---


Re: Discussion(New feature) Support Complex Data Type: Map in Carbon Data

2016-10-22 Thread cenyuhai
I think the map default delimiter should be the same with hive. 



--
View this message in context: 
http://apache-carbondata-mailing-list-archive.1130556.n5.nabble.com/Discussion-New-feature-Support-Complex-Data-Type-Map-in-Carbon-Data-tp1969p2239.html
Sent from the Apache CarbonData Mailing List archive mailing list archive at 
Nabble.com.