[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

[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

[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

[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

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 dat

[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

[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

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

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