Skip to content

[SPARK-35492][BUILD] Upgrade httpcore to 4.4.14#32638

Closed
vinodkc wants to merge 1 commit into
apache:masterfrom
vinodkc:br_build_upgrade_httpcore
Closed

[SPARK-35492][BUILD] Upgrade httpcore to 4.4.14#32638
vinodkc wants to merge 1 commit into
apache:masterfrom
vinodkc:br_build_upgrade_httpcore

Conversation

@vinodkc

@vinodkc vinodkc commented May 23, 2021

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

This PR aims to upgrade Apache HttpCore from 4.4.12 to 4.4.14.

Why are the changes needed?

Stability improvements in httpcore 4.4.14

  • Bug fix: Non-blocking TLSv1.3 connections can end up in an infinite event spin when closed concurrently by the local and the remote endpoints.
  • HTTPCORE-647: Non-blocking connection terminated due to 'java.io.IOException: Broken pipe' can enter an infinite loop flushing buffered output data.
  • PR Fix to Stage UI to display numbers on progress bar #201, HTTPCORE-634: Fix race condition in AbstractConnPool that can cause internal state
  • corruption
  • HTTPCORE-612: DefaultConnectionReuseStrategy incorrectly used int to represent Content-Length value
  • instead of long

Does this PR introduce any user-facing change?

No

How was this patch tested?

With Jenkins Tests

@github-actions github-actions Bot added the BUILD label May 23, 2021
@SparkQA

SparkQA commented May 23, 2021

Copy link
Copy Markdown

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43362/

@SparkQA

SparkQA commented May 23, 2021

Copy link
Copy Markdown

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43362/

@vinodkc vinodkc force-pushed the br_build_upgrade_httpcore branch from b166e41 to c700759 Compare May 23, 2021 11:57
@SparkQA

SparkQA commented May 23, 2021

Copy link
Copy Markdown

Test build #138840 has finished for PR 32638 at commit b166e41.

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

@SparkQA

SparkQA commented May 23, 2021

Copy link
Copy Markdown

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43367/

@SparkQA

SparkQA commented May 23, 2021

Copy link
Copy Markdown

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43367/

@SparkQA

SparkQA commented May 23, 2021

Copy link
Copy Markdown

Test build #138845 has finished for PR 32638 at commit c700759.

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

@dongjoon-hyun dongjoon-hyun left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM. Thank you, @vinodkc and @srowen .
Merged to maser.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants