[GitHub] spark pull request: Spark 8695

Github user piganesh commented on the pull request: https://github.com/apache/spark/pull/7397#issuecomment-123026246 Submitting a new pull request --- 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

[GitHub] spark pull request: Spark 8695

Github user piganesh commented on the pull request: https://github.com/apache/spark/pull/7397#issuecomment-122872705 OK I understood now. You had a comment on top to use .toDouble. Shall get to it ASAP. Sorry for the confusion. --- If your project is set up for it, you can reply to

[GitHub] spark pull request: Spark 8695

Github user piganesh commented on the pull request: https://github.com/apache/spark/pull/7397#issuecomment-122871746 I had made the changes to use ceiling instead of the hard coded greater than 5 check and submitted a pull request through ibmsoe github. I need to

[GitHub] spark pull request: Spark 8695

Github user piganesh commented on the pull request: https://github.com/apache/spark/pull/7397#issuecomment-122709018 Sean I did generate a new patch based on mengxr 's comment Namely while (numPartitions > scale + math.ceil(1.0 * numPartitions

[GitHub] spark pull request: Spark 8695

GitHub user piganesh opened a pull request: https://github.com/apache/spark/pull/7397 Spark 8695 You can merge this pull request into a Git repository by running: $ git pull https://github.com/ibmsoe/spark SPARK-8695 Alternatively you can review and apply these changes as

[GitHub] spark pull request: [SPARK-8695] [core] [WIP] TreeAggregation shou...

Github user piganesh commented on the pull request: https://github.com/apache/spark/pull/7168#issuecomment-120108884 From the abstract .. It seems you wanted to restrict it specifically for 5. Otherwise both while (numPartitions > scale + math.ceil(1.0 * numPartiti

[GitHub] spark pull request: [SPARK-8695] [core] [WIP] TreeAggregation shou...

Github user piganesh closed the pull request at: https://github.com/apache/spark/pull/7168 --- 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

[GitHub] spark pull request: [SPARK-8695] [core] [WIP] TreeAggregation shou...

GitHub user piganesh opened a pull request: https://github.com/apache/spark/pull/7168 [SPARK-8695] [core] [WIP] TreeAggregation shouldn't be triggered for … …5 partitions You can merge this pull request into a Git repository by running: $ git pull https://github.com/i

[GitHub] spark pull request: [SPARK-8130] - Made spark.files.useFetchCache ...

Github user piganesh commented on the pull request: https://github.com/apache/spark/pull/7051#issuecomment-115912368 Made spark.files.useFetchCache off by default --- 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

[GitHub] spark pull request: [SPARK-8130] - Made spark.files.useFetchCache ...

GitHub user piganesh opened a pull request: https://github.com/apache/spark/pull/7051 [SPARK-8130] - Made spark.files.useFetchCache off by default You can merge this pull request into a Git repository by running: $ git pull https://github.com/ibmsoe/spark SPARK-8130