Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/13238
---
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 user tdas commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221406335
Merging this to master and 2.0. Thanks for reviewing!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221402223
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221399106
**[Test build #3016 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3016/consoleFull)**
for PR 13238 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221379735
**[Test build #3016 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3016/consoleFull)**
for PR 13238 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221378746
**[Test build #59217 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59217/consoleFull)**
for PR 13238 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221378773
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221378770
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221376977
**[Test build #59217 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59217/consoleFull)**
for PR 13238 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221118006
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221118005
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221117792
**[Test build #59152 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59152/consoleFull)**
for PR 13238 at commit
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/13238#discussion_r64298276
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -186,6 +187,15 @@ case class DataSource(
val
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/13238#discussion_r64298319
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FileStreamSourceSuite.scala
---
@@ -430,20 +493,7 @@ class FileStreamSourceSuite extends
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/13238#discussion_r64297809
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FileStreamSourceSuite.scala
---
@@ -165,19 +177,32 @@ class FileStreamSourceSuite extends
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/13238#discussion_r64291854
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FileStreamSourceSuite.scala
---
@@ -140,6 +140,18 @@ class FileStreamSourceSuite extends
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/13238#discussion_r64291689
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FileStreamSourceSuite.scala
---
@@ -165,19 +177,32 @@ class FileStreamSourceSuite extends
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/13238#discussion_r64290699
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -186,6 +187,15 @@ case class DataSource(
Github user marmbrus commented on a diff in the pull request:
https://github.com/apache/spark/pull/13238#discussion_r64290791
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -186,6 +187,15 @@ case class DataSource(
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221087262
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221087266
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221086941
**[Test build #59151 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59151/consoleFull)**
for PR 13238 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221069144
**[Test build #59152 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59152/consoleFull)**
for PR 13238 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-221065433
**[Test build #59151 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59151/consoleFull)**
for PR 13238 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-220758077
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-220758078
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-220758049
**[Test build #59054 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59054/consoleFull)**
for PR 13238 at commit
Github user jaceklaskowski commented on a diff in the pull request:
https://github.com/apache/spark/pull/13238#discussion_r64127555
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FileStreamSourceSuite.scala
---
@@ -140,6 +140,18 @@ class FileStreamSourceSuite
Github user jaceklaskowski commented on a diff in the pull request:
https://github.com/apache/spark/pull/13238#discussion_r64127527
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSource.scala
---
@@ -186,6 +187,14 @@ case class DataSource(
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-220754573
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-220754572
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-220754537
**[Test build #59053 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59053/consoleFull)**
for PR 13238 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-220751877
**[Test build #59054 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59054/consoleFull)**
for PR 13238 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/13238#issuecomment-220751615
**[Test build #59053 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/59053/consoleFull)**
for PR 13238 at commit
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/13238#discussion_r64125854
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/FileStreamSourceSuite.scala
---
@@ -165,19 +177,32 @@ class FileStreamSourceSuite extends
GitHub user tdas opened a pull request:
https://github.com/apache/spark/pull/13238
[SPARK-15458][SQL][STREAMING] Disable schema inference for streaming
datasets on file streams
## What changes were proposed in this pull request?
If the user relies on the schema to be
36 matches
Mail list logo
