Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17001#discussion_r103046202
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -408,7 +408,15 @@ private[spark] class HiveExternalCatalog(co
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14830
Jenkins retest this please.
---
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 wis
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/14830
lets do a jenkins re-run just to make sure everything is up to date and
I'll try and get a final pass done soon. I think it would be good to improve
our examples to be closer to pep8 style for the s
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17001#discussion_r103045973
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -408,7 +408,15 @@ private[spark] class HiveExternalCatalog(co
Github user BryanCutler commented on a diff in the pull request:
https://github.com/apache/spark/pull/16774#discussion_r103045855
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/tuning/CrossValidatorSuite.scala ---
@@ -121,6 +121,33 @@ class CrossValidatorSuite
}
Github user merlintang commented on the issue:
https://github.com/apache/spark/pull/16965
@Yunni Yes, we can use the AND-OR to increase the possibility by having
more the numHashTables and numHashFunctions. For the further user extension, if
users have a hash function with lower poss
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16826
Merged build finished. Test PASSed.
---
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16826
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73442/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16826
**[Test build #73442 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73442/testReport)**
for PR 16826 at commit
[`fd11ee2`](https://github.com/apache/spark/commit/f
Github user devaraj-kavali commented on the issue:
https://github.com/apache/spark/pull/13072
Thanks @mgummelt for the confirmation, It throws SparkException with the
bug SPARK-15359/https://github.com/apache/spark/pull/13143.
---
If your project is set up for it, you can reply to th
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/15415
I agree that, if the set of rules is small (1-2 GB max), then collecting
and broadcasting it is best. But for larger sets of rules, we'd have to keep
it distributed.
I'm very surprised b
Github user squito commented on the issue:
https://github.com/apache/spark/pull/16930
thanks @kayousterhout !
---
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 s
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16930
---
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 ena
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/16930
I merged this into master
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16930
Merged build finished. Test PASSed.
---
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16930
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73437/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16930
**[Test build #73437 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73437/testReport)**
for PR 16930 at commit
[`ce6bf9a`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17043
**[Test build #73443 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73443/testReport)**
for PR 17043 at commit
[`e6b6dc1`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17043
Merged build finished. Test PASSed.
---
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17043
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73443/
Test PASSed.
---
Github user budde commented on a diff in the pull request:
https://github.com/apache/spark/pull/16944#discussion_r103027907
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/HiveSchemaInferenceSuite.scala
---
@@ -0,0 +1,192 @@
+/*
+ * Licensed to the Apache Softw
Github user budde commented on the issue:
https://github.com/apache/spark/pull/16944
The ```assert()``` statements added to ```setupCaseSensitiveTable()``` in
**HiveSchemaInferenceSuite** per earlier feedback got squashed somewhere in the
course of updating this PR. I've added them ba
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16944
**[Test build #73444 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73444/testReport)**
for PR 16944 at commit
[`e1ca7c8`](https://github.com/apache/spark/commit/e1
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/16944
@cloud-fan
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17043
**[Test build #73443 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73443/testReport)**
for PR 17043 at commit
[`e6b6dc1`](https://github.com/apache/spark/commit/e6
Github user xwu0226 commented on the issue:
https://github.com/apache/spark/pull/16592
@gatorsmile is this PR going to be merged soon, since the ALTER ADD column
PR #16626 also depends on this to create test cases for InMemoryCatalog. Thanks!
---
If your project is set up for it, you
Github user xwu0226 commented on a diff in the pull request:
https://github.com/apache/spark/pull/16626#discussion_r103024279
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -563,35 +574,47 @@ private[spark] class HiveExternalCatalog(con
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16826
**[Test build #73442 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73442/testReport)**
for PR 16826 at commit
[`fd11ee2`](https://github.com/apache/spark/commit/fd
Github user budde commented on the issue:
https://github.com/apache/spark/pull/16944
Thanks, @ericl. Is there anybody else you'd suggest pinging to take a look
at this and ultimately get it merged?
Re-pinging @viirya to review latest updates addressing his previous
feedback.
Github user marmbrus commented on the issue:
https://github.com/apache/spark/pull/16929
Hmm, I'm not sure we want to change this to a generator. I think that has
performance consequences as well as possibly being surprising. I would
probably make it possible to handle arrays (when t
Github user marmbrus commented on the issue:
https://github.com/apache/spark/pull/16929
/cc @brkyvz
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17032
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73440/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17032
Merged build finished. Test PASSed.
---
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17032
**[Test build #73440 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73440/testReport)**
for PR 17032 at commit
[`5beca69`](https://github.com/apache/spark/commit/5
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16892
---
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 ena
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16938#discussion_r103018002
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala
---
@@ -140,8 +140,8 @@ case class CreateDataSource
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/16892
I merged this into master -- thanks for the review @squito.
---
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
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16938#discussion_r103017698
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/createDataSourceTables.scala
---
@@ -140,8 +140,8 @@ case class CreateDataSource
Github user kayousterhout commented on a diff in the pull request:
https://github.com/apache/spark/pull/16892#discussion_r103017616
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/DAGSchedulerSuite.scala ---
@@ -2031,6 +2051,11 @@ class DAGSchedulerSuite extends SparkFunSu
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16976#discussion_r103014004
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVDataSource.scala
---
@@ -0,0 +1,238 @@
+/*
+ * Licensed to th
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16976#discussion_r103014118
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVDataSource.scala
---
@@ -0,0 +1,238 @@
+/*
+ * Licensed to th
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16976#discussion_r103015057
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/UnivocityParser.scala
---
@@ -269,3 +273,89 @@ private[csv] class Univoc
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16976#discussion_r103016252
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -961,56 +978,135 @@ class CSVSuite extends QueryTe
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16976#discussion_r103015384
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -961,56 +978,135 @@ class CSVSuite extends QueryTe
Github user kayousterhout commented on a diff in the pull request:
https://github.com/apache/spark/pull/16639#discussion_r103014993
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -400,8 +410,16 @@ private[spark] class Executor(
execBackend
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16976#discussion_r103012984
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/CodecStreams.scala
---
@@ -86,4 +88,11 @@ object CodecStreams {
.
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16976#discussion_r103012905
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/CodecStreams.scala
---
@@ -86,4 +88,11 @@ object CodecStreams {
.
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16976#discussion_r103015905
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
---
@@ -958,4 +975,77 @@ class CSVSuite extends QueryTest
Github user kayousterhout commented on a diff in the pull request:
https://github.com/apache/spark/pull/16639#discussion_r103015714
--- Diff:
core/src/main/scala/org/apache/spark/shuffle/FetchFailedException.scala ---
@@ -45,6 +50,12 @@ private[spark] class FetchFailedException(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13932
Merged build finished. Test PASSed.
---
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13932
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73435/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13932
**[Test build #73435 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73435/testReport)**
for PR 13932 at commit
[`a35f673`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17045
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73441/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17045
Merged build finished. Test PASSed.
---
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17045
**[Test build #73441 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73441/testReport)**
for PR 17045 at commit
[`6818ae0`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16630
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73439/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16630
Merged build finished. Test PASSed.
---
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16630
**[Test build #73439 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73439/testReport)**
for PR 16630 at commit
[`8e1c086`](https://github.com/apache/spark/commit/8
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/16987#discussion_r103014135
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FileStreamSourceSuite.scala
---
@@ -662,6 +663,101 @@ class FileStreamSourceSuite extends
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/16987#discussion_r103013496
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -243,13 +243,20 @@ case class DataSource(
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/16987#discussion_r103014293
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FileStreamSourceSuite.scala
---
@@ -662,6 +663,101 @@ class FileStreamSourceSuite extends
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/17031
@susanxhuynh I don't think it's worth documenting. It should be clear in
the logs, which should be where an operator turns if they notice no jobs are
launching.
---
If your project is set up for
Github user tritab commented on the issue:
https://github.com/apache/spark/pull/14807
It would probably be a good idea to get some unit tests for these
scenarios. Would anyone be willing to write some tests?
On Feb 24, 2017 10:13 AM, "roryodonnell" wrote:
> This c
Github user mgummelt commented on the issue:
https://github.com/apache/spark/pull/17031
@susanxhuynh Mesos/Spark integration tests:
https://github.com/typesafehub/mesos-spark-integration-tests. We run them as a
subset of DC/OS Spark integration tests:
https://github.com/mesosphere/s
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17045
**[Test build #73441 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73441/testReport)**
for PR 17045 at commit
[`6818ae0`](https://github.com/apache/spark/commit/68
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/16824
I'm slightly against the work to make this change happen and would rather
focus on some other Python PRs - I'm not sure it improves readability of the
test cases as much as planned but I'll defer to
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17032
**[Test build #73440 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73440/testReport)**
for PR 17032 at commit
[`5beca69`](https://github.com/apache/spark/commit/5b
Github user ericl commented on the issue:
https://github.com/apache/spark/pull/16944
LGTM
---
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
Github user budde commented on the issue:
https://github.com/apache/spark/pull/16944
@ericl: fixed the param doc string and tried to clean up
```createLogicalRelation()``` as you suggested.
---
If your project is set up for it, you can reply to this email and have your
reply appear o
Github user hhbyyh commented on the issue:
https://github.com/apache/spark/pull/15415
Hi @jkbradley After further performance comparison, I found using broadcast
would give much better performance for the transform.
I tested with some public data from http://fimi.ua.ac.be/data
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/17052
Thanks for doing this. I'm wondering if you can fix `isStreaming` instead.
We added it to be able to distinguish batch and streaming dataframes. However,
it doesn't work for batch DFs in a streaming
Github user wangmiao1981 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17032#discussion_r103004945
--- Diff: examples/src/main/r/ml/glm.R ---
@@ -25,12 +25,12 @@ library(SparkR)
sparkR.session(appName = "SparkR-ML-glm-example")
# $exam
Github user actuaryzhang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16630#discussion_r103004564
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GeneralizedLinearRegression.scala
---
@@ -1152,4 +1173,33 @@ class GeneralizedLinearRegr
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14412
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73431/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14412
Merged build finished. Test PASSed.
---
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14412
**[Test build #73431 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73431/testReport)**
for PR 14412 at commit
[`212baab`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16630
**[Test build #73439 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73439/testReport)**
for PR 16630 at commit
[`8e1c086`](https://github.com/apache/spark/commit/8e
Github user actuaryzhang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16630#discussion_r103003591
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/r/GeneralizedLinearRegressionWrapper.scala
---
@@ -99,37 +95,23 @@ private[r] object GeneralizedLin
Github user actuaryzhang commented on a diff in the pull request:
https://github.com/apache/spark/pull/16630#discussion_r103003515
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/regression/GeneralizedLinearRegression.scala
---
@@ -34,6 +35,7 @@ import org.apache.spark.rdd.RDD
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17043
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73438/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17043
Merged build finished. Test FAILed.
---
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17043
**[Test build #73438 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73438/testReport)**
for PR 17043 at commit
[`67e3c06`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17043
Merged build finished. Test PASSed.
---
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17043
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73436/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17043
**[Test build #73436 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73436/testReport)**
for PR 17043 at commit
[`129cfcd`](https://github.com/apache/spark/commit/1
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17051
---
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 ena
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/16594
---
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 ena
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17058
@HarshSharma8 this is not an improvement, please close this. This is not
the kind of PR that's worth submitting. See
http://spark.apache.org/contributing.html
---
If your project is set up for it,
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/16594
thanks, merging to master!
---
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 wi
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17051
thanks, merging to master!
---
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 wi
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/16930
LGTM assuming tests pass
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17043
**[Test build #73438 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73438/testReport)**
for PR 17043 at commit
[`67e3c06`](https://github.com/apache/spark/commit/67
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17043
**[Test build #73436 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73436/testReport)**
for PR 17043 at commit
[`129cfcd`](https://github.com/apache/spark/commit/12
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16930
**[Test build #73437 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73437/testReport)**
for PR 16930 at commit
[`ce6bf9a`](https://github.com/apache/spark/commit/ce
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14731
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73434/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14731
Merged build finished. Test PASSed.
---
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14731
**[Test build #73434 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73434/testReport)**
for PR 14731 at commit
[`724495b`](https://github.com/apache/spark/commit/7
Github user kayousterhout commented on the issue:
https://github.com/apache/spark/pull/15505
@witgo what's the status of this? I'd like to get this merged and am happy
to take this over if you don't have time to work on it.
---
If your project is set up for it, you can reply to this
Github user squito commented on a diff in the pull request:
https://github.com/apache/spark/pull/16892#discussion_r102996593
--- Diff:
core/src/test/scala/org/apache/spark/scheduler/DAGSchedulerSuite.scala ---
@@ -2031,6 +2051,11 @@ class DAGSchedulerSuite extends SparkFunSuite wit
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17049
---
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 ena
301 - 400 of 569 matches
Mail list logo