[GitHub] incubator-carbondata pull request #810: [CARBONDATA-935] Define PartitionInf...

2017-04-18 Thread lionelcao
GitHub user lionelcao opened a pull request: https://github.com/apache/incubator-carbondata/pull/810 [CARBONDATA-935] Define PartitionInfo Model modify schema.thrift modify TableSchema.java add new partition related classes You can merge this pull request into a Git

[GitHub] incubator-carbondata pull request #716: [CARBONDATA-840][WIP] improve limit ...

2017-03-31 Thread lionelcao
GitHub user lionelcao opened a pull request: https://github.com/apache/incubator-carbondata/pull/716 [CARBONDATA-840][WIP] improve limit query performance [Group By] Currently limit query will still scan all data first and limit in the last step. In carbon we can convert limit

[GitHub] incubator-carbondata pull request #661: remove shutdown dictionary server

2017-03-15 Thread lionelcao
GitHub user lionelcao opened a pull request: https://github.com/apache/incubator-carbondata/pull/661 remove shutdown dictionary server remove shutdown dictionary server You can merge this pull request into a Git repository by running: $ git pull https://github.com/lionelcao

[GitHub] incubator-carbondata pull request #655: [CARBONDATA-762] Change schemaName t...

2017-03-15 Thread lionelcao
GitHub user lionelcao opened a pull request: https://github.com/apache/incubator-carbondata/pull/655 [CARBONDATA-762] Change schemaName to databaseName, cubeName to table… Change schemaName to databaseName, cubeName to tableName You can merge this pull request into a Git

[GitHub] incubator-carbondata pull request #636: [CARBONDATA-741] Remove useless clas...

2017-03-08 Thread lionelcao
GitHub user lionelcao opened a pull request: https://github.com/apache/incubator-carbondata/pull/636 [CARBONDATA-741] Remove useless classes Remove useless classes VectorChunkRowIterator CarbonColumnVectorImpl You can merge this pull request into a Git repository

[GitHub] incubator-carbondata pull request #633: [CARBONDATA-739] Avoid creating mult...

2017-03-08 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/633#discussion_r105068561 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/collector/impl/DictionaryBasedResultCollector.java --- @@ -37,6 +34,8

[GitHub] incubator-carbondata issue #629: [CARBONDATA-740] add logger for total rows ...

2017-03-08 Thread lionelcao
Github user lionelcao commented on the issue: https://github.com/apache/incubator-carbondata/pull/629 @ravipesala this PR is closed but not merged yet. I can't reopen it. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-carbondata pull request #633: [CARBONDATA-739] Avoid creating mult...

2017-03-08 Thread lionelcao
GitHub user lionelcao opened a pull request: https://github.com/apache/incubator-carbondata/pull/633 [CARBONDATA-739] Avoid creating multipul instance of directDictionaryGenerator Avoid creating multipul instance of directDictionaryGenerator You can merge this pull request

[GitHub] incubator-carbondata issue #629: [CARBONDATA-740] add logger for total rows ...

2017-03-07 Thread lionelcao
Github user lionelcao commented on the issue: https://github.com/apache/incubator-carbondata/pull/629 @ravipesala Added step name, please check. --- 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

[GitHub] incubator-carbondata issue #628: [CARBONDATA-743] Remove redundant CarbonFil...

2017-03-07 Thread lionelcao
Github user lionelcao commented on the issue: https://github.com/apache/incubator-carbondata/pull/628 Hi @ravipesala , I compared those two files before commit and I think the difference has no impact to the functionality. Any ideas? --- If your project is set up for it, you can

[GitHub] incubator-carbondata pull request #629: [CARBONDATA-740] add logger for tota...

2017-03-07 Thread lionelcao
GitHub user lionelcao opened a pull request: https://github.com/apache/incubator-carbondata/pull/629 [CARBONDATA-740] add logger for total rows processed add logger for total rows processed You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-carbondata pull request #628: [CARBONDATA-743] Remove redundant Ca...

2017-03-07 Thread lionelcao
GitHub user lionelcao opened a pull request: https://github.com/apache/incubator-carbondata/pull/628 [CARBONDATA-743] Remove redundant CarbonFilters file Remove redundant CarbonFilters file in spark2 and keep the one in spark common You can merge this pull request into a Git

[GitHub] incubator-carbondata pull request #625: [CARBONDATA-743] Remove redundant Ca...

2017-03-07 Thread lionelcao
Github user lionelcao closed the pull request at: https://github.com/apache/incubator-carbondata/pull/625 --- 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

[GitHub] incubator-carbondata pull request #625: [CARBONDATA-743] Remove redundant Ca...

2017-03-06 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/625#discussion_r104575573 --- Diff: integration/spark2/src/main/scala/org/apache/carbondata/spark/CarbonFilters.scala --- @@ -1,397 +0,0 @@ -/* - * Licensed

[GitHub] incubator-carbondata pull request #625: [CARBONDATA-743] Remove redundant Ca...

2017-03-06 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/625#discussion_r104575582 --- Diff: integration/spark2/src/main/scala/org/apache/carbondata/spark/CarbonFilters.scala --- @@ -1,397 +0,0 @@ -/* - * Licensed

[GitHub] incubator-carbondata pull request #625: [CARBONDATA-743] Remove redundant Ca...

2017-03-06 Thread lionelcao
Github user lionelcao commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/625#discussion_r104578046 --- Diff: integration/spark2/src/main/scala/org/apache/carbondata/spark/CarbonFilters.scala --- @@ -1,397 +0,0 @@ -/* - * Licensed