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

2016-09-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-carbondata/pull/161 --- 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 f

[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 CarbonMerge

[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 CarbonMerger

[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 CarbonMerge

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

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

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

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

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

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

[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_r79283767 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/Compactor.scala --- @@ -69,7 +69,9 @@ object Compactor {

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

2016-09-17 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/161#discussion_r79282917 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/Compactor.scala --- @@ -69,7 +69,9 @@ object Compactor {

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

2016-09-16 Thread ravikiran23
GitHub user ravikiran23 opened a pull request: https://github.com/apache/incubator-carbondata/pull/161 [CARBONDATA-246] compaction is wrong in case if last segment is not assigned to an executor. PROBLEM: if during compaction of 4 loads, for any executor if only first 3 loa