[GitHub] incubator-carbondata issue #791: [CARBONDATA-920] Updated useful-tips-on-car...
Github user sgururajshetty commented on the issue: https://github.com/apache/incubator-carbondata/pull/791 LGTM @chenliang613 kindly reivew --- 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 issue #794: [CARBONDATA-857] Added Alter Table document...
Github user sgururajshetty commented on the issue: https://github.com/apache/incubator-carbondata/pull/794 LGTM @chenliang613 kindly review --- 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 issue #611: [CARBONDATA-731] Enhance and correct quick ...
Github user sgururajshetty commented on the issue: https://github.com/apache/incubator-carbondata/pull/611 LGTM. The files are also formatted and reviewed for better understanding. --- 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 issue #601: [CARBONDATA-709]corrected doc for bucketing
Github user sgururajshetty commented on the issue: https://github.com/apache/incubator-carbondata/pull/601 LGTM --- 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 issue #602: [CARBONDATA-710] Updated the FAQs and Troub...
Github user sgururajshetty commented on the issue: https://github.com/apache/incubator-carbondata/pull/602 LGTM --- 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 issue #582: [CARBONDATA-687] Added documentation for ne...
Github user sgururajshetty commented on the issue: https://github.com/apache/incubator-carbondata/pull/582 @chenliang613 please review and let us know any changes required. --- 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 #550: [CARBONDATA-624] Complete CarbonData...
GitHub user sgururajshetty opened a pull request: https://github.com/apache/incubator-carbondata/pull/550 [CARBONDATA-624] Complete CarbonData document in md files 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. --- You can merge this pull request into a Git repository by running: $ git pull https://github.com/sgururajshetty/incubator-carbondata master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-carbondata/pull/550.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 #550 commit fab13bd15fd8196c25889c4e5734773555e0b384 Author: sgururajshetty Date: 2017-01-09T12:06:03Z Document update for UID Two new DML commands added as part of the UID UPDATE CARBON TABLE DELETE RECORDS from CARBON TABLE commit 6408094a9ba1dc7c52cfddec4b1f1d9ff4a0431d Author: sgururajshetty Date: 2017-01-10T05:58:46Z Merge branch 'master' into master commit 9466537be49f4a06e75f10c56e3ea8c7d00d7eb5 Author: sgururajshetty Date: 2017-01-10T06:02:45Z Hyperlink fixed Some reference information removed. commit bf61c596a533b1db9a1a6a777d3de30994b1ad4b Author: sgururajshetty Date: 2017-01-10T06:14:35Z Example refined commit 59baa3163a7d63eef4ec2cef606cc88ea90c421f Author: sgururajshetty Date: 2017-01-18T11:26:28Z [CARBONDATA-624] Complete CarbonData document in md files Complete CarbonData documentation in md files --- 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 issue #510: Document update for UID
Github user sgururajshetty commented on the issue: https://github.com/apache/incubator-carbondata/pull/510 @jackylk the review comments is been incorporated in the new md files. --- 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 #510: Document update for UID
Github user sgururajshetty closed the pull request at: https://github.com/apache/incubator-carbondata/pull/510 --- 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 issue #510: Document update for UID
Github user sgururajshetty commented on the issue: https://github.com/apache/incubator-carbondata/pull/510 @jackylk have fixed the issue. 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 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 #510: Document update for UID
GitHub user sgururajshetty opened a pull request: https://github.com/apache/incubator-carbondata/pull/510 Document update for UID Two new DML commands added as part of the UID UPDATE CARBON TABLE DELETE RECORDS from CARBON TABLE 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. --- You can merge this pull request into a Git repository by running: $ git pull https://github.com/sgururajshetty/incubator-carbondata master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-carbondata/pull/510.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 #510 commit fab13bd15fd8196c25889c4e5734773555e0b384 Author: sgururajshetty Date: 2017-01-09T12:06:03Z Document update for UID Two new DML commands added as part of the UID UPDATE CARBON TABLE DELETE RECORDS from CARBON TABLE --- 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. ---
