Skip to content

Conversation

@srowen
Copy link
Member

@srowen srowen commented May 2, 2016

What changes were proposed in this pull request?

Update Jetty 8.1 to the latest 2016/02 release, from a 2013/10 release, for security and bug fixes. This does not resolve the JIRA necessarily, as it's still worth considering an update to 9.3.

How was this patch tested?

Jenkins tests

@SparkQA
Copy link

SparkQA commented May 2, 2016

Test build #57531 has finished for PR 12842 at commit 81eb44e.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented May 2, 2016

Test build #2957 has finished for PR 12842 at commit 81eb44e.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@vanzin
Copy link
Contributor

vanzin commented May 2, 2016

LGTM, but the bug id in the title is misspelled. Updating to Jetty 9, IIRC, is not currently an option - I believe it's jdk8-only.

@srowen srowen changed the title [SPAR-14897] [CORE] Upgrade Jetty to latest version of 8 [SPARK-14897] [CORE] Upgrade Jetty to latest version of 8 May 2, 2016
@srowen
Copy link
Member Author

srowen commented May 2, 2016

Oops, fixed. It looks like 9.3 requires JDK 8, but 9.2 doesn't. Still, probably worth doing this when JDK 8 is required all at once. For now maybe moving to 8.1.x's latest is sufficient.

asfgit pushed a commit that referenced this pull request May 3, 2016
## What changes were proposed in this pull request?

Update Jetty 8.1 to the latest 2016/02 release, from a 2013/10 release, for security and bug fixes. This does not resolve the JIRA necessarily, as it's still worth considering an update to 9.3.

## How was this patch tested?

Jenkins tests

Author: Sean Owen <[email protected]>

Closes #12842 from srowen/SPARK-14897.

(cherry picked from commit 57ac7c1)
Signed-off-by: Sean Owen <[email protected]>
@asfgit asfgit closed this in 57ac7c1 May 3, 2016
@srowen
Copy link
Member Author

srowen commented May 3, 2016

Merged this much to master/2.0/1.6

asfgit pushed a commit that referenced this pull request May 3, 2016
Update Jetty 8.1 to the latest 2016/02 release, from a 2013/10 release, for security and bug fixes. This does not resolve the JIRA necessarily, as it's still worth considering an update to 9.3.

Jenkins tests

Author: Sean Owen <[email protected]>

Closes #12842 from srowen/SPARK-14897.

(cherry picked from commit 57ac7c1)
Signed-off-by: Sean Owen <[email protected]>
@srowen srowen deleted the SPARK-14897 branch May 3, 2016 13:00
zzcclp pushed a commit to zzcclp/spark that referenced this pull request May 4, 2016
Update Jetty 8.1 to the latest 2016/02 release, from a 2013/10 release, for security and bug fixes. This does not resolve the JIRA necessarily, as it's still worth considering an update to 9.3.

Jenkins tests

Author: Sean Owen <[email protected]>

Closes apache#12842 from srowen/SPARK-14897.

(cherry picked from commit 57ac7c1)
Signed-off-by: Sean Owen <[email protected]>
(cherry picked from commit 2db19a3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants