Github user bsikander commented on the issue:
https://github.com/apache/spark/pull/21816
Thank you.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21816
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21816
**[Test build #4392 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4392/testReport)**
for PR 21816 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21816
**[Test build #4392 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4392/testReport)**
for PR 21816 at commit
Github user bsikander commented on the issue:
https://github.com/apache/spark/pull/21816
@srowen Can you have another look?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21816
Ping @srinathshankar @ericl again for comments? I don't know this well, but
seems like a low risk change at worst.
---
-
To
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21816
**[Test build #4357 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4357/testReport)**
for PR 21816 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21816
**[Test build #4357 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4357/testReport)**
for PR 21816 at commit
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/21816
Agree, though I'm just going to run them again
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user bsikander commented on the issue:
https://github.com/apache/spark/pull/21816
Failing tests don't seem to be related to this change.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21816
**[Test build #4356 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4356/testReport)**
for PR 21816 at commit
Github user bsikander commented on the issue:
https://github.com/apache/spark/pull/21816
>> So is there no behavior change if your master specifies only one master?
If user specifies only 1 master then there is no behavioral change.
---
Github user bsikander commented on the issue:
https://github.com/apache/spark/pull/21816
@srowen I removed the extra .toString.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user bsikander commented on the issue:
https://github.com/apache/spark/pull/21816
@srowen even though it is unlikely but it could happen. I have updated the
code.
---
-
To unsubscribe, e-mail:
Github user bsikander commented on the issue:
https://github.com/apache/spark/pull/21816
>> What might break if we make this change? I think that's what any other
reviewer cares abou
To the best of my knowledge. Nothing should break.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21816
cc @srinathshankar @ericl
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21816
That could help, yes.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user bsikander commented on the issue:
https://github.com/apache/spark/pull/21816
@vanzin ok, how can I find the right reviewer for this change? Posting on
dev mailing list?
---
-
To unsubscribe, e-mail:
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21816
I know you pinged me, but I'm really not familiar with how Spark standalone
is deployed, and I don't even know who's maintaining this code these days...
---
Github user bsikander commented on the issue:
https://github.com/apache/spark/pull/21816
Could some please have a look at this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user bsikander commented on the issue:
https://github.com/apache/spark/pull/21816
could someone please have a look at this change?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user bsikander commented on the issue:
https://github.com/apache/spark/pull/21816
Sorry for dropping message again, but could someone please have a look at
this change. This is a problem that I am having constantly and a validation
from senior committers will be really
Github user bsikander commented on the issue:
https://github.com/apache/spark/pull/21816
@holdenk somehow, Jenkins didn't execute the tests.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/21816
Jenkins OK to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user bsikander commented on the issue:
https://github.com/apache/spark/pull/21816
@vanzin Could you please have a look on this change?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21816
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21816
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21816
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
28 matches
Mail list logo