Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/5449
---
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 enab
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/5449#issuecomment-136894778
@twinkle-sachdeva can you close this PR?
---
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 proje
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5449#issuecomment-121073540
Can one of the admins verify this patch?
---
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 pr
Github user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/5449#issuecomment-117186673
Not a problem. Can you close this when you have a chance and I'll post
comment to see if someone can take over. thanks.
---
If your project is set up for it, you ca
Github user twinkle-sachdeva commented on the pull request:
https://github.com/apache/spark/pull/5449#issuecomment-117001196
Hi Tom,
It will be good, if somebody else can take up. I am not doing well, it
might get delayed a bit more.
Sorry for the inconvenience cau
Github user twinkle-sachdeva commented on the pull request:
https://github.com/apache/spark/pull/5449#issuecomment-116946917
Hi Tom,
I will do that in one or two days.
Thanks,
Twinkle
On Mon, Jun 29, 2015 at 7:44 PM, Tom Graves
wrote:
> @twin
Github user tgravescs commented on the pull request:
https://github.com/apache/spark/pull/5449#issuecomment-116689661
@twinkle-sachdeva do you have time to address at Sandy's comments
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHu
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r30287086
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -119,7 +130,27 @@ private[yarn] class YarnAllocator(
def getN
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r30287118
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -83,6 +84,9 @@ private[yarn] class YarnAllocator(
private var execu
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r30286942
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -119,7 +130,27 @@ private[yarn] class YarnAllocator(
def getN
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r30286962
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -119,7 +130,27 @@ private[yarn] class YarnAllocator(
def getN
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r30286851
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -94,6 +98,13 @@ private[yarn] class YarnAllocator(
// Additional me
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r30286771
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -20,6 +20,7 @@ package org.apache.spark.deploy.yarn
import java.util.
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r30286779
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -119,7 +130,27 @@ private[yarn] class YarnAllocator(
def getN
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r30286760
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala ---
@@ -59,6 +59,9 @@ private[spark] class ApplicationMaster(
private
Github user twinkle-sachdeva commented on the pull request:
https://github.com/apache/spark/pull/5449#issuecomment-101248742
Hi,
Can somebody please review the change.
Regards,
Twinkle
On Fri, Apr 10, 2015 at 1:57 PM, UCB AMPLab
wrote:
> Can
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5449#issuecomment-96767452
Can one of the admins verify this patch?
---
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 pro
Github user twinkle-sachdeva commented on the pull request:
https://github.com/apache/spark/pull/5449#issuecomment-96515946
Hi @srowen ,
Please review the changes.
Thanks,
---
If your project is set up for it, you can reply to this email and have your
reply appear o
Github user twinkle-sachdeva commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r28394987
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -94,6 +98,14 @@ private[yarn] class YarnAllocator(
// Ad
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r28278175
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -94,6 +98,14 @@ private[yarn] class YarnAllocator(
// Additional me
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r28233526
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -94,6 +98,14 @@ private[yarn] class YarnAllocator(
// Additional m
Github user twinkle-sachdeva commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r28224409
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -94,6 +98,14 @@ private[yarn] class YarnAllocator(
// Ad
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r28218090
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -94,6 +98,14 @@ private[yarn] class YarnAllocator(
// Additional me
Github user sryza commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r28217902
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -119,7 +131,27 @@ private[yarn] class YarnAllocator(
def getN
Github user twinkle-sachdeva commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r28215620
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -94,6 +98,14 @@ private[yarn] class YarnAllocator(
// Ad
Github user twinkle-sachdeva commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r28215570
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocator.scala ---
@@ -119,7 +131,27 @@ private[yarn] class YarnAllocator(
Github user twinkle-sachdeva commented on a diff in the pull request:
https://github.com/apache/spark/pull/5449#discussion_r28215509
--- Diff:
yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala ---
@@ -59,6 +59,10 @@ private[spark] class ApplicationMaster(
27 matches
Mail list logo