[GitHub] incubator-carbondata pull request #207: [CARBONDATA-283] VT enhancement for ...

2016-11-04 Thread ManoharVanam
Github user ManoharVanam commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/207#discussion_r86523287 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonTableStatusUtil.java --- @@ -0,0 +1,92

[GitHub] incubator-carbondata pull request #207: [CARBONDATA-283] VT enhancement for ...

2016-11-04 Thread ManoharVanam
Github user ManoharVanam commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/207#discussion_r86523222 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonTableStatusUtil.java --- @@ -0,0 +1,92

[GitHub] incubator-carbondata pull request #207: [CARBONDATA-283] VT enhancement for ...

2016-10-03 Thread ManoharVanam
GitHub user ManoharVanam opened a pull request: https://github.com/apache/incubator-carbondata/pull/207 [CARBONDATA-283] VT enhancement for data retention concurrent scenarios why raise this PR ? 1 . Handling concurrency scenarios for retention feature 2. Added test cases

[GitHub] incubator-carbondata pull request #194: [CARBONDATA-270] Double data type va...

2016-09-23 Thread ManoharVanam
Github user ManoharVanam commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/194#discussion_r80212063 --- Diff: core/src/main/java/org/apache/carbondata/scan/filter/FilterUtil.java --- @@ -1401,8 +1401,7 @@ public static void logError

[GitHub] incubator-carbondata pull request #192: [CARBONDATA-269]change delete segmen...

2016-09-23 Thread ManoharVanam
Github user ManoharVanam commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/192#discussion_r80060992 --- Diff: integration/spark/src/main/scala/org/apache/spark/sql/CarbonSqlParser.scala --- @@ -1181,23 +1182,21 @@ class CarbonSqlParser

[GitHub] incubator-carbondata pull request #167: [CARBONDATA-250] Throw exception and...

2016-09-17 Thread ManoharVanam
GitHub user ManoharVanam opened a pull request: https://github.com/apache/incubator-carbondata/pull/167 [CARBONDATA-250] Throw exception and fail the data load if provided MAXCOLUMNS value is not proper [Problem] : If user executes data load query with improper maxcolumns value

[GitHub] incubator-carbondata pull request #165: [WIP] Changed long to Bigint

2016-09-17 Thread ManoharVanam
GitHub user ManoharVanam opened a pull request: https://github.com/apache/incubator-carbondata/pull/165 [WIP] Changed long to Bigint Changed long to Bigint Testing scenarios You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-carbondata pull request #164: [WIP] Changed Long to BigInt

2016-09-17 Thread ManoharVanam
Github user ManoharVanam closed the pull request at: https://github.com/apache/incubator-carbondata/pull/164 --- 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 #164: [WIP] Changed Long to BigInt

2016-09-17 Thread ManoharVanam
GitHub user ManoharVanam opened a pull request: https://github.com/apache/incubator-carbondata/pull/164 [WIP] Changed Long to BigInt Changed Long to BigInt You can merge this pull request into a Git repository by running: $ git pull https://github.com/ManoharVanam/incubator

[GitHub] incubator-carbondata pull request #156: [CARBONDATA-244] Load and delete seg...

2016-09-16 Thread ManoharVanam
Github user ManoharVanam commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/156#discussion_r79222066 --- Diff: processing/src/main/java/org/apache/carbondata/lcm/status/SegmentStatusManager.java --- @@ -306,13 +330,18 @@ private Integer

[GitHub] incubator-carbondata pull request #156: [CARBONDATA-244] Load and delete seg...

2016-09-16 Thread ManoharVanam
Github user ManoharVanam commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/156#discussion_r79217913 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataRDDFactory.scala --- @@ -1077,19 +1077,38 @@ object

[GitHub] incubator-carbondata pull request #156: [WIP] Added table Status lock while ...

2016-09-16 Thread ManoharVanam
Github user ManoharVanam commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/156#discussion_r79119039 --- Diff: processing/src/main/java/org/apache/carbondata/lcm/status/SegmentStatusManager.java --- @@ -306,13 +330,17 @@ private Integer

[GitHub] incubator-carbondata pull request #156: [WIP] Added table Status lock while ...

2016-09-16 Thread ManoharVanam
Github user ManoharVanam commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/156#discussion_r79119024 --- Diff: processing/src/main/java/org/apache/carbondata/lcm/status/SegmentStatusManager.java --- @@ -246,15 +247,22 @@ private Integer

[GitHub] incubator-carbondata pull request #156: [WIP] Added table Status lock while ...

2016-09-14 Thread ManoharVanam
GitHub user ManoharVanam opened a pull request: https://github.com/apache/incubator-carbondata/pull/156 [WIP] Added table Status lock while deleting the segments Added table Status lock while deleting the segments throwing exception if lock is not available You can merge

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

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