[GitHub] incubator-carbondata pull request #814: [WIP] Fixed table not found exceptio...

2017-04-18 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/814 [WIP] Fixed table not found exception in rename table after lock acquire fa… You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-carbondata issue #802: [CARBONDATA-926] Changed max columns from s...

2017-04-15 Thread kunal642
Github user kunal642 commented on the issue: https://github.com/apache/incubator-carbondata/pull/802 retest this please --- 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

[GitHub] incubator-carbondata pull request #802: [CARBONDATA-926] Changed max columns...

2017-04-15 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/802 [CARBONDATA-926] Changed max columns from static value to configurable value in Load DDL Changes done:- 1. Earlier the value for max columns that can be read by the csv

[GitHub] incubator-carbondata issue #777: [CARBONDATA-863] Separated column schema ge...

2017-04-14 Thread kunal642
Github user kunal642 commented on the issue: https://github.com/apache/incubator-carbondata/pull/777 @gvramana please review this --- 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 #777: [CARBONDATA-863] Moved AlterTableAdd...

2017-04-13 Thread kunal642
Github user kunal642 closed the pull request at: https://github.com/apache/incubator-carbondata/pull/777 --- 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 #777: [CARBONDATA-863] Moved AlterTableAdd...

2017-04-13 Thread kunal642
GitHub user kunal642 reopened a pull request: https://github.com/apache/incubator-carbondata/pull/777 [CARBONDATA-863] Moved AlterTableAddColumnRDD call to AlterTableCommands Moved call to AlterTableAddColumnRDD to AlterTableCommands Refactored code Added test case You can

[GitHub] incubator-carbondata issue #777: [CARBONDATA-863] Moved AlterTableAddColumnR...

2017-04-13 Thread kunal642
Github user kunal642 commented on the issue: https://github.com/apache/incubator-carbondata/pull/777 retest this please --- 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

[GitHub] incubator-carbondata pull request #780: [CARBONDATA-892] Fixed NullPointerEx...

2017-04-11 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/780 [CARBONDATA-892] Fixed NullPointerException for 2nd level subquery You can merge this pull request into a Git repository by running: $ git pull https://github.com/kunal642

[GitHub] incubator-carbondata issue #777: [CARBONDATA-863] Moved AlterTableAddColumnR...

2017-04-11 Thread kunal642
Github user kunal642 commented on the issue: https://github.com/apache/incubator-carbondata/pull/777 retest this please --- 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

[GitHub] incubator-carbondata pull request #777: [CARBONDATA-863] Moved AlterTableAdd...

2017-04-11 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/777 [CARBONDATA-863] Moved AlterTableAddColumnRDD call to AlterTableCommands You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-carbondata issue #771: [CARBONDATA-883] Fixed cast exception for i...

2017-04-10 Thread kunal642
Github user kunal642 commented on the issue: https://github.com/apache/incubator-carbondata/pull/771 @gvramana 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 feature

[GitHub] incubator-carbondata pull request #771: [CARBONDATA-883] Fixed cast exceptio...

2017-04-09 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/771 [CARBONDATA-883] Fixed cast exception for integer data types in RestructureBasedVectorResultCollector The following issues are fixed in this PR:- 1. In RestructureUtil, int and

[GitHub] incubator-carbondata pull request #753: [CARBONDATA-877] Fixed ArrayIndexOut...

2017-04-06 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/753 [CARBONDATA-877] Fixed ArrayIndexOutOfBoundException for ColumnarVectorWrapper Row id was being incremented with the column number and eventually the index on which we had to put

[GitHub] incubator-carbondata issue #727: [CARBONDATA-846] Added support to revert ch...

2017-04-06 Thread kunal642
Github user kunal642 commented on the issue: https://github.com/apache/incubator-carbondata/pull/727 @manishgupta88 ok --- 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

[GitHub] incubator-carbondata issue #717: [CARBONDATA-839] Fixed issue with meta lock...

2017-04-05 Thread kunal642
Github user kunal642 commented on the issue: https://github.com/apache/incubator-carbondata/pull/717 @ravipesala Added comments. 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

[GitHub] incubator-carbondata pull request #717: [CARBONDATA-839] Fixed issue with me...

2017-04-05 Thread kunal642
Github user kunal642 closed the pull request at: https://github.com/apache/incubator-carbondata/pull/717 --- 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 #717: [CARBONDATA-839] Fixed issue with me...

2017-04-05 Thread kunal642
GitHub user kunal642 reopened a pull request: https://github.com/apache/incubator-carbondata/pull/717 [CARBONDATA-839] Fixed issue with meta lock not getting deleted for rename table The following things are fixed in this patch: 1. Meta lock location has been changed from

[GitHub] incubator-carbondata issue #717: [CARBONDATA-839] Fixed issue with meta lock...

2017-04-05 Thread kunal642
Github user kunal642 commented on the issue: https://github.com/apache/incubator-carbondata/pull/717 retest this please --- 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

[GitHub] incubator-carbondata pull request #717: [CARBONDATA-839] Fixed issue with me...

2017-04-05 Thread kunal642
Github user kunal642 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/717#discussion_r109868326 --- Diff: core/src/main/java/org/apache/carbondata/core/locks/LocalFileLock.java --- @@ -70,7 +71,8

[GitHub] incubator-carbondata pull request #724: [CARBONDATA-847] Fixed NullPointerEx...

2017-04-05 Thread kunal642
Github user kunal642 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/724#discussion_r109861944 --- Diff: core/src/main/java/org/apache/carbondata/core/scan/collector/impl/DictionaryBasedVectorResultCollector.java --- @@ -51,8 +51,8

[GitHub] incubator-carbondata pull request #717: [CARBONDATA-839] Fixed issue with me...

2017-04-05 Thread kunal642
Github user kunal642 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/717#discussion_r109859846 --- Diff: core/src/main/java/org/apache/carbondata/core/locks/LocalFileLock.java --- @@ -70,7 +71,8

[GitHub] incubator-carbondata pull request #727: [CARBONDATA-846] Added support to re...

2017-04-04 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/727 [CARBONDATA-846] Added support to revert changes if query fails You can merge this pull request into a Git repository by running: $ git pull https://github.com/kunal642

[GitHub] incubator-carbondata pull request #724: [CARBONDATA-847] Fixed NullPointerEx...

2017-04-03 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/724 [CARBONDATA-847] Fixed NullPointerException for DictionaryBasedVectorResultCollector during alter table You can merge this pull request into a Git repository by running: $ git

[GitHub] incubator-carbondata pull request #717: [CARBONDATA-839] Fixed issue with me...

2017-03-31 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/717 [CARBONDATA-839] Fixed issue with meta lock not getting deleted for rename table The following things are fixed in this patch: 1. Meta lock location has been changed from

[GitHub] incubator-carbondata issue #695: [CARBONDATA-696] Fixed Null Pointer Excepti...

2017-03-23 Thread kunal642
Github user kunal642 commented on the issue: https://github.com/apache/incubator-carbondata/pull/695 @kumarvishal09 review this please --- 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 #695: Fixed Null Pointer Exception for dou...

2017-03-23 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/695 Fixed Null Pointer Exception for double data type without fraction Fixed null pointer exception for double data type Removed unused methods You can merge this pull request into a

[GitHub] incubator-carbondata issue #678: [CARBONDATA-798] Added support to rename ba...

2017-03-21 Thread kunal642
Github user kunal642 commented on the issue: https://github.com/apache/incubator-carbondata/pull/678 @gvramana Changes have been done --- 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 issue #656: [CARBONDATA-773] Fixed multiple DictionaryS...

2017-03-15 Thread kunal642
Github user kunal642 commented on the issue: https://github.com/apache/incubator-carbondata/pull/656 @ravipesala changes made. 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

[GitHub] incubator-carbondata pull request #656: [CARBONDATA-773] Fixed multiple Dict...

2017-03-15 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/656 [CARBONDATA-773] Fixed multiple DictionaryServer instances issue Fixed the issue where multiple dictionary server instances were being created for parallel load. You can merge this

[GitHub] incubator-carbondata pull request #381: [CARBONDATA-485] Refactored code for...

2017-03-13 Thread kunal642
Github user kunal642 closed the pull request at: https://github.com/apache/incubator-carbondata/pull/381 --- 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 #381: [CARBONDATA-485] Refactored code for...

2016-12-02 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/381 [CARBONDATA-485] Refactored code for DataGraphExecuter Removed unused parameters Added constants You can merge this pull request into a Git repository by running: $ git

[GitHub] incubator-carbondata pull request #371: [CARBONDATA-472] Added unit test for...

2016-11-29 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/371 [CARBONDATA-472] Added unit test for cache package Added unit test for CarbonLRUCacheTest Added unit test for ColumnDictionaryChunkIteratorTest Added unit test for

[GitHub] incubator-carbondata pull request #349: [CARBONDATA-446] Added unit test for...

2016-11-24 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/349 [CARBONDATA-446] Added unit test for scan.collector.impl package Added unit test for DictionaryBasedResultCollector Added unit test for RawBasedResultCollector You can merge this

[GitHub] incubator-carbondata pull request #346: [Carbondata-377] Add unit test for c...

2016-11-23 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/346 [Carbondata-377] Add unit test for core.cache.dictionary package Added unit test for DictionaryColumnUniqueIdentifier Added unit test for ReverseDictionary Added unit test for

[GitHub] incubator-carbondata pull request #342: added unit test for FilterScanner

2016-11-22 Thread kunal642
GitHub user kunal642 opened a pull request: https://github.com/apache/incubator-carbondata/pull/342 added unit test for FilterScanner Added unit test for FilterScanner You can merge this pull request into a Git repository by running: $ git pull https://github.com/kunal642