Github user srowen closed the pull request at:
https://github.com/apache/spark/pull/10446
---
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 srowen commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-168390287
Merged 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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-167985723
**[Test build #48475 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48475/consoleFull)**
for PR 10446 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-167985799
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/10446#issuecomment-167985797
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 srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10446#discussion_r48593961
--- Diff: core/src/main/scala/org/apache/spark/SparkHadoopWriter.scala ---
@@ -143,6 +141,12 @@ class SparkHadoopWriter(jobConf: JobConf)
taskContext
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-167971026
I'm pushing an update with those changes. I found a similar one in
`FileBasedWriteAheadLogWriter` and one more in `FsHistoryProvider` regarding
checking for safe mode.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-167971529
**[Test build #48475 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48475/consoleFull)**
for PR 10446 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-168004825
**[Test build #2266 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2266/consoleFull)**
for PR 10446 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-167987565
**[Test build #2266 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2266/consoleFull)**
for PR 10446 at commit
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10446#discussion_r48581372
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/WriterContainer.scala
---
@@ -24,10 +24,9 @@ import
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10446#discussion_r48581350
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/SqlNewHadoopRDD.scala
---
@@ -26,10 +26,10 @@ import
Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-167907581
LGTM; there's at least one extra possible cleanup, but feel free to punt on
that one.
---
If your project is set up for it, you can reply to this email and have your
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10446#discussion_r48581268
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/EventLoggingListener.scala ---
@@ -97,7 +97,7 @@ private[spark] class EventLoggingListener(
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-167867032
Yea I think this one is fine.
(I haven't looked at the detailed changes though)
---
If your project is set up for it, you can reply to this email and have your
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10446#discussion_r48581127
--- Diff: core/src/main/scala/org/apache/spark/SparkHadoopWriter.scala ---
@@ -143,6 +141,12 @@ class SparkHadoopWriter(jobConf: JobConf)
taskContext
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/10446#discussion_r48581178
--- Diff: core/src/main/scala/org/apache/spark/rdd/NewHadoopRDD.scala ---
@@ -26,11 +26,11 @@ import org.apache.hadoop.conf.{Configurable,
Configuration}
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-167773896
Pinging folks like @vanzin @pwendell @rxin @sarutak
I don't think this one is controversial, as it just follows from removing
older Hadoop support and cleans up the
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-167566796
**[Test build #48366 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48366/consoleFull)**
for PR 10446 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-167590960
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/10446#issuecomment-167590880
**[Test build #48366 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48366/consoleFull)**
for PR 10446 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-167590962
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/10446#issuecomment-166923370
**[Test build #48243 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48243/consoleFull)**
for PR 10446 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-166960152
**[Test build #2253 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2253/consoleFull)**
for PR 10446 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-166947653
**[Test build #48243 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48243/consoleFull)**
for PR 10446 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-166947721
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/10446#issuecomment-166947723
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/10446#issuecomment-166982259
**[Test build #2253 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2253/consoleFull)**
for PR 10446 at commit
GitHub user srowen opened a pull request:
https://github.com/apache/spark/pull/10446
[SPARK-12481] [CORE] [STREAMING] [SQL] Remove usage of Hadoop deprecated
APIs and reflection that supported 1.x
Remove use of deprecated Hadoop APIs now that 2.2+ is required
You can merge this
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-166884922
**[Test build #48240 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48240/consoleFull)**
for PR 10446 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10446#issuecomment-166888672
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/10446#issuecomment-166888673
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/10446#issuecomment-166888644
**[Test build #48240 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48240/consoleFull)**
for PR 10446 at commit
33 matches
Mail list logo