[GitHub] incubator-carbondata pull request #140: [CARBONDATA-227] In block distributi...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/incubator-carbondata/pull/140


---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-carbondata pull request #140: [CARBONDATA-227] In block distributi...

[gvramana]

Github user gvramana commented on a diff in the pull request:

https://github.com/apache/incubator-carbondata/pull/140#discussion_r77950753
  
--- Diff: 
integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonScanRDD.scala
 ---
@@ -73,7 +73,7 @@ class CarbonScanRDD[V: ClassTag](
 baseStoreLocation: String)
   extends RDD[V](sc, Nil) with Logging {
 
-  val defaultParallelism = sc.defaultParallelism
+  var defaultParallelism = sc.defaultParallelism
--- End diff --

now member is not required, make it a private val.


---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-carbondata pull request #140: [CARBONDATA-227] In block distributi...

[mohammadshahidkhan]

GitHub user mohammadshahidkhan opened a pull request:

https://github.com/apache/incubator-carbondata/pull/140

[CARBONDATA-227] In block distribution parralelism is decided initial…

In block distribution parralelism is decided initially and not re 
initialized after requesting new executors. Due to this task per node 
initialization is getting wrong.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mohammadshahidkhan/incubator-carbondata 
parrallelism_Calculation_issue

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-carbondata/pull/140.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #140


commit c1c49fcc0da754459b42911089d77f57219c8a81
Author: mohammadshahidkhan 
Date:   2016-09-08T05:07:49Z

[CARBONDATA-227] In block distribution parralelism is decided initially and 
not re initialized after requesting new executors. Due to this task per node 
initialization is getting wrong.




---
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 feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---