[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-12 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/spark/pull/12916


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-12 Thread srowen
Github user srowen commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218855583
  
Merged to master/2.0


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-12 Thread bomeng
Github user bomeng commented on a diff in the pull request:

https://github.com/apache/spark/pull/12916#discussion_r63060249
  
--- Diff: yarn/pom.xml ---
@@ -102,6 +102,10 @@
   org.eclipse.jetty
   jetty-servlet
 
+
+  org.eclipse.jetty
+  jetty-servlets
+
--- End diff --

yes, we need both, they have different content.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-12 Thread srowen
Github user srowen commented on a diff in the pull request:

https://github.com/apache/spark/pull/12916#discussion_r63040583
  
--- Diff: yarn/pom.xml ---
@@ -102,6 +102,10 @@
   org.eclipse.jetty
   jetty-servlet
 
+
+  org.eclipse.jetty
+  jetty-servlets
+
--- End diff --

Looks good to me. Just confirming there really are both "jetty-servlet" and 
"jetty-servlets" artifacts that we need both of?


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218756911
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58479/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-12 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218756907
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-12 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218756643
  
**[Test build #58479 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58479/consoleFull)**
 for PR 12916 at commit 
[`37f00a7`](https://github.com/apache/spark/commit/37f00a71a0e266f2f686dbb245fa73bbbfc0f82f).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-12 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218731481
  
**[Test build #58479 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58479/consoleFull)**
 for PR 12916 at commit 
[`37f00a7`](https://github.com/apache/spark/commit/37f00a71a0e266f2f686dbb245fa73bbbfc0f82f).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-12 Thread bomeng
Github user bomeng commented on a diff in the pull request:

https://github.com/apache/spark/pull/12916#discussion_r63003988
  
--- Diff: core/pom.xml ---
@@ -125,12 +125,17 @@
   jetty-servlet
   compile
 
+
+  org.eclipse.jetty
+  jetty-servlets
+  compile
+
 

[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-12 Thread srowen
Github user srowen commented on a diff in the pull request:

https://github.com/apache/spark/pull/12916#discussion_r62996834
  
--- Diff: core/pom.xml ---
@@ -125,12 +125,17 @@
   jetty-servlet
   compile
 
+
+  org.eclipse.jetty
+  jetty-servlets
+  compile
+
 

[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-11 Thread bomeng
Github user bomeng commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218668164
  
@srowen sorry for the late reply, I did not notice it. I have run the mvn 
dependency:tree and only javax.servlet-api 3.1.0 is listed, so it should be 
fine.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-11 Thread bomeng
Github user bomeng commented on a diff in the pull request:

https://github.com/apache/spark/pull/12916#discussion_r62968602
  
--- Diff: core/pom.xml ---
@@ -125,12 +125,17 @@
   jetty-servlet
   compile
 
+
+  org.eclipse.jetty
+  jetty-servlets
+  compile
+
 

[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218462330
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218462335
  
Test PASSed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58367/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-11 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218462065
  
**[Test build #58367 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58367/consoleFull)**
 for PR 12916 at commit 
[`ca949f9`](https://github.com/apache/spark/commit/ca949f9a9a5a0668d9c0d7311e34a0a6e07a5177).
 * This patch passes all tests.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-11 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218436248
  
**[Test build #58367 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58367/consoleFull)**
 for PR 12916 at commit 
[`ca949f9`](https://github.com/apache/spark/commit/ca949f9a9a5a0668d9c0d7311e34a0a6e07a5177).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-11 Thread srowen
Github user srowen commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218435879
  
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218435474
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-11 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218435475
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58361/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-11 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218435301
  
**[Test build #58361 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58361/consoleFull)**
 for PR 12916 at commit 
[`ca949f9`](https://github.com/apache/spark/commit/ca949f9a9a5a0668d9c0d7311e34a0a6e07a5177).
 * This patch **fails Spark unit tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-11 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218416953
  
**[Test build #58361 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58361/consoleFull)**
 for PR 12916 at commit 
[`ca949f9`](https://github.com/apache/spark/commit/ca949f9a9a5a0668d9c0d7311e34a0a6e07a5177).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-11 Thread srowen
Github user srowen commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218416369
  
@bomeng this is looking nice. The last worry I have is about the 
javax.servlet dependency. I remember we had significant problems with servlet 
containers bringing in their own copy under their own artifact name, and 
conflicting with the main `javax.servlet` artifact. I think we ended up 
excluding everything except Jetty's copy. That seems to be changed in this PR, 
which could be fine. But if you would try `mvn dependency:tree` and grep for 
any "javax.+servlet" artifacts, and see if anything whose version looks like 
it's less than 3.1.0 is still included. (Likewise maybe we can remove some old 
excludes of stuff like this, but I'm not as worried.)

If there are no other servlet APIs coming in, I'm down with this.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-11 Thread srowen
Github user srowen commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218416401
  
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 wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-10 Thread holdenk
Github user holdenk commented on a diff in the pull request:

https://github.com/apache/spark/pull/12916#discussion_r62784093
  
--- Diff: core/pom.xml ---
@@ -125,12 +125,17 @@
   jetty-servlet
   compile
 
+
+  org.eclipse.jetty
+  jetty-servlets
+  compile
+
 

[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-10 Thread bomeng
Github user bomeng commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-218340455
  
@srowen Finally I've got it working. Servlet and Derby were upgraded as 
well due to requirement of Jetty. Please review.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-217060384
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57829/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-217060383
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-04 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-217060376
  
**[Test build #57829 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57829/consoleFull)**
 for PR 12916 at commit 
[`68a8144`](https://github.com/apache/spark/commit/68a8144493f2e17fa1f730c9e6b748a0fec2aec2).
 * This patch **fails to build**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-04 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-217059838
  
**[Test build #57829 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57829/consoleFull)**
 for PR 12916 at commit 
[`68a8144`](https://github.com/apache/spark/commit/68a8144493f2e17fa1f730c9e6b748a0fec2aec2).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-217044265
  
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-04 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-217044262
  
**[Test build #57820 has 
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57820/consoleFull)**
 for PR 12916 at commit 
[`adba870`](https://github.com/apache/spark/commit/adba870ed702d4bd53292f240785fbd86484bb9d).
 * This patch **fails build dependency tests**.
 * This patch merges cleanly.
 * This patch adds no public classes.


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-217044267
  
Test FAILed.
Refer to this link for build results (access rights to CI server needed): 
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57820/
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
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-04 Thread SparkQA
Github user SparkQA commented on the pull request:

https://github.com/apache/spark/pull/12916#issuecomment-217043865
  
**[Test build #57820 has 
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57820/consoleFull)**
 for PR 12916 at commit 
[`adba870`](https://github.com/apache/spark/commit/adba870ed702d4bd53292f240785fbd86484bb9d).


---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org



[GitHub] spark pull request: [SPARK-14897] [SQL] upgrade to jetty 9.2.16

2016-05-04 Thread bomeng
GitHub user bomeng opened a pull request:

https://github.com/apache/spark/pull/12916

[SPARK-14897] [SQL] upgrade to jetty 9.2.16

## What changes were proposed in this pull request?

Since Jetty 8 is EOL (end of life) and has critical security issue 
[http://www.securityweek.com/critical-vulnerability-found-jetty-web-server], I 
think upgrading to 9 is necessary. I am using latest 9.2 since 9.3 requires 
Java 8+.

## How was this patch tested?

Manual test and current test cases should cover it.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/bomeng/spark SPARK-14897

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/spark/pull/12916.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #12916


commit adba870ed702d4bd53292f240785fbd86484bb9d
Author: bomeng 
Date:   2016-05-04T23:55:54Z

upgrade to jetty 9.2.16




---
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 enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org