[GitHub] incubator-carbondata pull request #796: [CARBONDATA-808] Added Example for P...

2017-04-14 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/796 [CARBONDATA-808] Added Example for Presto You can merge this pull request into a Git repository by running: $ git pull https://github.com/PallaviSingh1992/incubator

[GitHub] incubator-carbondata pull request #794: [CARBONDATA-857] Added Alter Table d...

2017-04-13 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/794 [CARBONDATA-857] Added Alter Table documentation Added Documentation for following alter commands: Add Column Drop Columns Change Data Type Also

[GitHub] incubator-carbondata pull request #763: [CARBONDATA-857] Added Documentation...

2017-04-13 Thread PallaviSingh1992
Github user PallaviSingh1992 closed the pull request at: https://github.com/apache/incubator-carbondata/pull/763 --- 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 #763: [CARBONDATA-857] Added Documentation...

2017-04-07 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/763 [CARBONDATA-857] Added Documentation for Alter Table Added Documentation for following alter commands: - Add Column - Drop Columns - Change Data Type Also

[GitHub] incubator-carbondata pull request #658: [CARBONDATA-775]Updated Date DataTyp...

2017-03-16 Thread PallaviSingh1992
Github user PallaviSingh1992 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/658#discussion_r106408192 --- Diff: docs/supported-data-types-in-carbondata.md --- @@ -22,17 +22,18 @@ CarbonData supports the following data types

[GitHub] incubator-carbondata issue #658: [CARBONDATA-775]Updated Date DataType in Da...

2017-03-15 Thread PallaviSingh1992
Github user PallaviSingh1992 commented on the issue: https://github.com/apache/incubator-carbondata/pull/658 @chenliang613 please 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

[GitHub] incubator-carbondata pull request #658: [CARBONDATA-775]Updated Date DataTyp...

2017-03-15 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/658 [CARBONDATA-775]Updated Date DataType in DataTypes supported by CarbonData Added the support for Date Data Type and re-structured the file. You can merge this pull request

[GitHub] incubator-carbondata issue #614: [CARBONDATA-714]Documented how to handle ba...

2017-03-01 Thread PallaviSingh1992
Github user PallaviSingh1992 commented on the issue: https://github.com/apache/incubator-carbondata/pull/614 @chenliang613 please 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

[GitHub] incubator-carbondata pull request #614: [CarbonData-714]Documented how to ha...

2017-02-27 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/614 [CarbonData-714]Documented how to handle bad records You can merge this pull request into a Git repository by running: $ git pull https://github.com/PallaviSingh1992

[GitHub] incubator-carbondata pull request #602: [CARBONDATA-710] Updated the FAQs an...

2017-02-22 Thread PallaviSingh1992
Github user PallaviSingh1992 closed the pull request at: https://github.com/apache/incubator-carbondata/pull/602 --- 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 #602: [CARBONDATA-710] Updated the FAQs an...

2017-02-21 Thread PallaviSingh1992
Github user PallaviSingh1992 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/602#discussion_r102399607 --- Diff: docs/faq.md --- @@ -18,30 +18,124 @@ --> # FAQs -* **Auto Compaction not Working** -

[GitHub] incubator-carbondata issue #602: [CARBONDATA-710] Updated the FAQs and Troub...

2017-02-21 Thread PallaviSingh1992
Github user PallaviSingh1992 commented on the issue: https://github.com/apache/incubator-carbondata/pull/602 @chenliang613 please 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

[GitHub] incubator-carbondata pull request #602: [CARBONDATA-710] Updated the FAQs an...

2017-02-21 Thread PallaviSingh1992
Github user PallaviSingh1992 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/602#discussion_r102384069 --- Diff: docs/faq.md --- @@ -17,30 +17,128 @@ under the License. --> +

[GitHub] incubator-carbondata pull request #602: [CARBONDATA-710] Updated the FAQs an...

2017-02-16 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/602 [CARBONDATA-710] Updated the FAQs and Troubleshooting Added content to FAQs and Troubleshooting You can merge this pull request into a Git repository by running: $ git

[GitHub] incubator-carbondata pull request #587: [CARBONDATA-694] Updated Quick Start

2017-02-06 Thread PallaviSingh1992
Github user PallaviSingh1992 closed the pull request at: https://github.com/apache/incubator-carbondata/pull/587 --- 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 issue #587: [CARBONDATA-694] Updated Quick Start

2017-02-05 Thread PallaviSingh1992
Github user PallaviSingh1992 commented on the issue: https://github.com/apache/incubator-carbondata/pull/587 @chenliang613 please 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

[GitHub] incubator-carbondata pull request #587: [CARBONDATA-694] Updated Quick Start

2017-02-05 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/587 [CARBONDATA-694] Updated Quick Start You can merge this pull request into a Git repository by running: $ git pull https://github.com/PallaviSingh1992/incubator

[GitHub] incubator-carbondata pull request #582: [CARBONDATA-687] Added documentation...

2017-01-30 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/582 [CARBONDATA-687] Added documentation for new features in the DML, DDL Section and Troubleshooting You can merge this pull request into a Git repository by running

[GitHub] incubator-carbondata pull request #575: [CARBONDATA-682] Added License to Fl...

2017-01-26 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/575 [CARBONDATA-682] Added License to FloatDataTypeTestCase.scala and DateTypeTest.scala You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-carbondata pull request #574: [CARBONDATA-682] Added License to Fl...

2017-01-26 Thread PallaviSingh1992
Github user PallaviSingh1992 closed the pull request at: https://github.com/apache/incubator-carbondata/pull/574 --- 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 #574: [CARBONDATA-682] Added License to Fl...

2017-01-26 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/574 [CARBONDATA-682] Added License to FloatDataTypeTestCase.scala and DateTypeTest.scala You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-carbondata pull request #568: Fix docs issues

2017-01-23 Thread PallaviSingh1992
Github user PallaviSingh1992 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/568#discussion_r97266651 --- Diff: docs/configuration-parameters.md --- @@ -142,7 +142,7 @@ This section provides the details of all the configurations

[GitHub] incubator-carbondata pull request #568: Fix docs issues

2017-01-23 Thread PallaviSingh1992
Github user PallaviSingh1992 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/568#discussion_r97266540 --- Diff: docs/configuration-parameters.md --- @@ -142,7 +142,7 @@ This section provides the details of all the configurations

[GitHub] incubator-carbondata pull request #568: Fix docs issues

2017-01-22 Thread PallaviSingh1992
Github user PallaviSingh1992 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/568#discussion_r97265941 --- Diff: docs/configuration-parameters.md --- @@ -132,7 +132,7 @@ This section provides the details of all the configurations

[GitHub] incubator-carbondata pull request #563: [CARBONDATA-670] Added MD Files

2017-01-20 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/563 [CARBONDATA-670] Added MD Files Added following MD Files 1. File Structure 2. Data Types And Updated the Overview Section You can merge this pull request

[GitHub] incubator-carbondata pull request #370: [CARBONDATA-457] Added Unit Test cas...

2017-01-16 Thread PallaviSingh1992
Github user PallaviSingh1992 closed the pull request at: https://github.com/apache/incubator-carbondata/pull/370 --- 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 #428: [CARBONDATA-529] - Added Unit Test c...

2017-01-16 Thread PallaviSingh1992
Github user PallaviSingh1992 closed the pull request at: https://github.com/apache/incubator-carbondata/pull/428 --- 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 #336: [CARBONDATA-426] replace if else wit...

2016-12-13 Thread PallaviSingh1992
Github user PallaviSingh1992 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/336#discussion_r92158292 --- Diff: integration/spark-common/src/main/java/org/apache/carbondata/spark/merger/CarbonCompactionUtil.java --- @@ -142,20 +142,15

[GitHub] incubator-carbondata pull request #428: [CARBONDATA-529] - Added Unit Test c...

2016-12-13 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/428 [CARBONDATA-529] - Added Unit Test cases for processing.newflow.parser package You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] incubator-carbondata pull request #336: [CARBONDATA-426] replace if else wit...

2016-12-01 Thread PallaviSingh1992
Github user PallaviSingh1992 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/336#discussion_r90590085 --- Diff: integration/spark-common/src/main/java/org/apache/carbondata/spark/merger/CarbonCompactionUtil.java --- @@ -142,20 +142,14

[GitHub] incubator-carbondata pull request #370: [CARBONDATA-457] Added Unit Test cas...

2016-11-29 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/370 [CARBONDATA-457] Added Unit Test cases for core.writer package You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-carbondata pull request #361: [CARBONDATA-457] - Added Unit Test C...

2016-11-28 Thread PallaviSingh1992
Github user PallaviSingh1992 closed the pull request at: https://github.com/apache/incubator-carbondata/pull/361 --- 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 #361: [CARBONDATA-457] - Add Unit Tests Fo...

2016-11-28 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/361 [CARBONDATA-457] - Add Unit Tests For core.writer package You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-carbondata pull request #337: [CARBONDATA-428] Removed Redundant C...

2016-11-22 Thread PallaviSingh1992
Github user PallaviSingh1992 closed the pull request at: https://github.com/apache/incubator-carbondata/pull/337 --- 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 #337: [CARBONDATA-428] Removed Redundant C...

2016-11-21 Thread PallaviSingh1992
Github user PallaviSingh1992 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/337#discussion_r89041271 --- Diff: core/src/main/java/org/apache/carbondata/core/cache/CacheProvider.java --- @@ -84,14 +84,13 @@ public static CacheProvider

[GitHub] incubator-carbondata pull request #337: [CARBONDATA-428] Removed Redundant C...

2016-11-21 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/337 [CARBONDATA-428] Removed Redundant Condition Checks Remove the redundant conditional checking of dictionaryCacheAlreadyExists(cacheType) in CacheProvider. You can

[GitHub] incubator-carbondata pull request #336: [CARBONDATA-426] replace if else wit...

2016-11-20 Thread PallaviSingh1992
GitHub user PallaviSingh1992 opened a pull request: https://github.com/apache/incubator-carbondata/pull/336 [CARBONDATA-426] replace if else with conditional operator Replaced the if-else with Conditional operator. Reduced the number of variables used. Reduced the boiler