[GitHub] incubator-carbondata pull request #245: [CARBONDATA-320] problem during drop...

2016-10-17 Thread ravikiran23
GitHub user ravikiran23 opened a pull request: https://github.com/apache/incubator-carbondata/pull/245 [CARBONDATA-320] problem during drop of table when all datanodes are down. Problem : when all the data nodes are down and user executes drop table. then drop table

[GitHub] incubator-carbondata pull request #205: [CARBONDATA-281]Vt enhancement for t...

2016-09-30 Thread ravikiran23
GitHub user ravikiran23 opened a pull request: https://github.com/apache/incubator-carbondata/pull/205 [CARBONDATA-281]Vt enhancement for the Life cycle management module why raise this PR ? Improving the test cases for the LCM module. Test cases for the scenarios like

[GitHub] incubator-carbondata pull request #161: [CARBONDATA-246] compaction is wrong...

2016-09-17 Thread ravikiran23
Github user ravikiran23 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/161#discussion_r79290314 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonMergerRDD.scala --- @@ -280,6 +277,25 @@ class

[GitHub] incubator-carbondata pull request #161: [CARBONDATA-246] compaction is wrong...

2016-09-17 Thread ravikiran23
Github user ravikiran23 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/161#discussion_r79290299 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonMergerRDD.scala --- @@ -259,6 +253,9 @@ class

[GitHub] incubator-carbondata pull request #161: [CARBONDATA-246] compaction is wrong...

2016-09-17 Thread ravikiran23
Github user ravikiran23 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/161#discussion_r79290197 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonMergerRDD.scala --- @@ -102,6 +102,11 @@ class

[GitHub] incubator-carbondata pull request #168: [CARBONDATA-251] making the auto com...

2016-09-17 Thread ravikiran23
Github user ravikiran23 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/168#discussion_r79287261 --- Diff: core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java --- @@ -920,7 +920,7 @@ * Default

[GitHub] incubator-carbondata pull request #160: [CARBONDATA-245] Actual Exception is...

2016-09-17 Thread ravikiran23
Github user ravikiran23 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/160#discussion_r79284972 --- Diff: core/src/main/java/org/apache/carbondata/core/writer/ThriftWriter.java --- @@ -80,6 +80,17 @@ public void open() throws

[GitHub] incubator-carbondata pull request #160: [CARBONDATA-245] Actual Exception is...

2016-09-16 Thread ravikiran23
GitHub user ravikiran23 opened a pull request: https://github.com/apache/incubator-carbondata/pull/160 [CARBONDATA-245] Actual Exception is getting lost in case of data dictionary file generation. PROBLEM : during data load if any exception comes when the dictionary file

[GitHub] incubator-carbondata pull request #149: [CARBONDATA-234]wrong message gettin...

2016-09-16 Thread ravikiran23
Github user ravikiran23 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/149#discussion_r79223519 --- Diff: integration/spark/src/main/java/org/apache/carbondata/integration/spark/merger/CarbonCompactionUtil.java --- @@ -210,9 +210,12

[GitHub] incubator-carbondata pull request #149: [CARBONDATA-234]wrong message gettin...

2016-09-15 Thread ravikiran23
Github user ravikiran23 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/149#discussion_r78961659 --- Diff: integration/spark/src/main/java/org/apache/carbondata/integration/spark/merger/CarbonCompactionUtil.java --- @@ -246,6 +251,9

[GitHub] incubator-carbondata pull request #149: [CARBONDATA-234]wrong message gettin...

2016-09-15 Thread ravikiran23
Github user ravikiran23 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/149#discussion_r78961612 --- Diff: integration/spark/src/main/java/org/apache/carbondata/integration/spark/merger/CarbonCompactionUtil.java --- @@ -210,10 +211,14

[GitHub] incubator-carbondata pull request #155: [CARBONDATA-239] Handling the except...

2016-09-14 Thread ravikiran23
GitHub user ravikiran23 opened a pull request: https://github.com/apache/incubator-carbondata/pull/155 [CARBONDATA-239] Handling the exception cases in compaction. Why raise this pr? If a compaction is triggered by the user on table1 , and other requests will go to queue

[GitHub] incubator-carbondata pull request #149: wrong message getting displayed whil...

2016-09-11 Thread ravikiran23
GitHub user ravikiran23 opened a pull request: https://github.com/apache/incubator-carbondata/pull/149 wrong message getting displayed while compaction. Why raise this pr? for each compaction the error message "compaction request file can not be deleted " is pri

[GitHub] incubator-carbondata pull request #116: [CARBONDATA-198] System level lock f...

2016-09-01 Thread ravikiran23
Github user ravikiran23 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/116#discussion_r77131782 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataRDDFactory.scala --- @@ -333,7 +531,7 @@ object

[GitHub] incubator-carbondata pull request #116: [CARBONDATA-198] System level lock f...

2016-09-01 Thread ravikiran23
Github user ravikiran23 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/116#discussion_r77131071 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataRDDFactory.scala --- @@ -297,15 +366,144 @@ object

[GitHub] incubator-carbondata pull request #116: [CARBONDATA-198] System level lock f...

2016-09-01 Thread ravikiran23
Github user ravikiran23 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/116#discussion_r77130739 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataRDDFactory.scala --- @@ -297,15 +366,144 @@ object

[GitHub] incubator-carbondata pull request #116: [CARBONDATA-198] System level lock f...

2016-09-01 Thread ravikiran23
Github user ravikiran23 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/116#discussion_r77119210 --- Diff: core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java --- @@ -904,6 +904,27

[GitHub] incubator-carbondata pull request #92: [CARBONDATA-176] Deletion of compacte...

2016-08-29 Thread ravikiran23
Github user ravikiran23 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/92#discussion_r76619356 --- Diff: processing/src/main/java/org/apache/carbondata/lcm/status/SegmentStatusManager.java --- @@ -449,6 +457,12 @@ public void

[GitHub] incubator-carbondata pull request #92: [CARBONDATA-176] Deletion of compacte...

2016-08-29 Thread ravikiran23
Github user ravikiran23 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/92#discussion_r76606724 --- Diff: processing/src/main/java/org/apache/carbondata/lcm/status/SegmentStatusManager.java --- @@ -410,18 +410,26 @@ public void

[GitHub] incubator-carbondata pull request #92: [CARBONDATA-176] Deletion of compacte...

2016-08-29 Thread ravikiran23
Github user ravikiran23 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/92#discussion_r76581081 --- Diff: processing/src/main/java/org/apache/carbondata/lcm/status/SegmentStatusManager.java --- @@ -410,6 +410,14 @@ public void

[GitHub] incubator-carbondata pull request #92: [CARBONDATA-176] Deletion of compacte...

2016-08-24 Thread ravikiran23
GitHub user ravikiran23 opened a pull request: https://github.com/apache/incubator-carbondata/pull/92 [CARBONDATA-176] Deletion of compacted segment is not allowed. once the segment is compacted then if user selects the compacted segment to delete then no need to delete the segment

[GitHub] incubator-carbondata pull request #86: [CARBONDATA-170] Delete the lock file...

2016-08-23 Thread ravikiran23
GitHub user ravikiran23 opened a pull request: https://github.com/apache/incubator-carbondata/pull/86 [CARBONDATA-170] Delete the lock file once the unlocking is done. Issue : The lock files which are used for locking purpose need to get deleted once lock is unlocked