[GitHub] incubator-carbondata pull request #308: [WIP] In DropCarbonTable flow, Metad...

2016-11-09 Thread nareshpr
GitHub user nareshpr opened a pull request:

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

[WIP] In DropCarbonTable flow, Metadata lock should be acquired only if 
tab…

Issue : In drop table flow, we acquiring metadata lock even if table not 
exist which is creating table folder and creating meta.lock file.
Solution : Check and acquire lock only if carbon table exist.

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

$ git pull https://github.com/nareshpr/incubator-carbondata droptablefix

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

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


commit 20813ccbffae1aa48bdef069721aeeebc550bc04
Author: nareshpr 
Date:   2016-11-09T12:43:36Z

In DropCarbonTable flow, Metadata lock should be acquired only if table 
exist




---
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 #289: [WIP] Drop Database Cascade command ...

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

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

[WIP] Drop Database Cascade command supported in Carbon.

Issue : Currently drop database cascade command is restricted in Carbon
Solution : Implemented code to support below changes
1) Create DB folder in carbon store when user triggers Create database 
command
2) Removing DB folder from carbon store when user triggers Drop database 
command
3) Removing Tables & DB folder from carbon store when user triggers Drop 
database cascade command

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

$ git pull https://github.com/nareshpr/incubator-carbondata dropdbcascade

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

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


commit c392427096ea5f8d7ad608e75d7ef9298f705935
Author: nareshpr 
Date:   2016-11-03T09:23:03Z

Drop Database Cascade command to be supported in Carbon.




---
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 #166: [WIP] As LongType.simpleString in sp...

2016-09-17 Thread nareshpr
GitHub user nareshpr opened a pull request:

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

[WIP] As LongType.simpleString in spark is "bigint", Carbon will convert 
Long to BigInt

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [ ] Make sure the PR title is formatted like:
   `[CARBONDATA-] Description of pull request`
 - [ ] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [ ] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).
 - [ ] Testing done
 
Please provide details on 
- Whether new unit test cases have been added or why no new tests 
are required?
- What manual testing you have done?
- Any additional information to help reviewers in testing this 
change.
 
 - [ ] For large changes, please consider breaking it into sub-tasks under 
an umbrella JIRA. 
 
---

…bigint" which will be shown in describe

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

$ git pull https://github.com/nareshpr/incubator-carbondata 
bigintfromlongfix

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

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


commit de281a664d78fb635a83b651fed8ecc9885991d3
Author: nareshpr 
Date:   2016-09-17T13:04:13Z

In Carbon, BigInt is only supported, Spark LongType.simpleString is 
"bigint" which will be shown in describe




---
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 #136: [WIP] Removed unused code in CarbonS...

2016-09-07 Thread nareshpr
GitHub user nareshpr opened a pull request:

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

[WIP] Removed unused code in CarbonSqlParser



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

$ git pull https://github.com/nareshpr/incubator-carbondata 
removingunusedcode

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

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






---
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 #105: [CARBONDATA-189] Drop database casca...

2016-08-29 Thread nareshpr
GitHub user nareshpr opened a pull request:

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

[CARBONDATA-189] Drop database cascade should be restricted

Reason: Carbondata will not support "Drop database dbname cascade"
When user provides this command, carbon will validate and give below error 
message "Unsupported cascade operation in drop database command"

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

$ git pull https://github.com/nareshpr/incubator-carbondata 
restrictdropdbcascade

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

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


commit 295694793a6a3f0616b6ac18527bef86c6644ee5
Author: nareshpr 
Date:   2016-08-29T09:55:40Z

drop database cascade should be restricted




---
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 #102: [CARBONDATA-186] Except compaction a...

2016-08-29 Thread nareshpr
GitHub user nareshpr opened a pull request:

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

[CARBONDATA-186] Except compaction all other alter operations on carbon 
table will be unsupported.

Reason: As Carbon table will not support alter operations except 
compaction, all the alter operations on carbon table should be skipped and 
error message should be displayed as "Unsupported alter operation on carbon 
table"
Whereas if the alter operation is on hive table, it should be transferred 
to hive for performing the operation.

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

$ git pull https://github.com/nareshpr/incubator-carbondata 
altertableunsupportedoperations

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

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


commit 8ef206511fd1b8d83de63308fc338f85688f6451
Author: nareshpr 
Date:   2016-08-29T06:57:23Z

Alter operations on carbon table will be unsupported.




---
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 #99: [CARBONDATA-184] Fixed special char d...

2016-08-26 Thread nareshpr
GitHub user nareshpr opened a pull request:

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

[CARBONDATA-184] Fixed special char delimiter for complex data.

Reason : When complex types data is having ^ delimiter, data is not getting 
loaded properly. When the data is queried, output is NULL
eg: array and complex data level1 delimiter is '^' 
actual data is 
john^carter^justin
data is not getting split and not getting loaded & when queried, its give 
NULL value.

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

$ git pull https://github.com/nareshpr/incubator-carbondata 
complexdataspecialdelimitersupport

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

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


commit 03b744827480c096a9036ec56baec41d382ef5b1
Author: nareshpr 
Date:   2016-08-26T09:10:27Z

Fixed special char delimiter for complex data.




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