Skip to content

Conversation

@atokhy
Copy link

@atokhy atokhy commented Apr 7, 2016

What changes were proposed in this pull request?

This splits commons.httpclient.version from commons.httpcore.version, since these two versions do not necessarily have to be the same. This change may follow up with an up-to-date version of the httpclient/httpcore libraries.

The latest 4.3.x httpclient version as of writing is 4.3.6 and the latest 4.3.x httpcore version as of writing is 4.3.3. This change would be a prerequisite for potentially moving to this new bugfix version.

How was this patch tested?

no version change was made for httpclient/httpcore versions
mvn package

@atokhy
Copy link
Author

atokhy commented Apr 7, 2016

Reviewing related pull requests, this might conflict with:

#10887

@srowen
Copy link
Member

srowen commented Apr 7, 2016

Although I think it's rare you'd want to set these differently, it's not a big deal to make this change, yes.

@srowen
Copy link
Member

srowen commented Apr 7, 2016

Jenkins test this please

@SparkQA
Copy link

SparkQA commented Apr 8, 2016

Test build #55258 has finished for PR 12245 at commit c0ad429.

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

@JoshRosen
Copy link
Contributor

This seems fine to me as well.

@srowen
Copy link
Member

srowen commented Apr 8, 2016

Merged to master

@asfgit asfgit closed this in 583b5e0 Apr 8, 2016
@atokhy atokhy deleted the pull-request branch April 30, 2016 02:48
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.

5 participants