You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jclouds.apache.org by Andrew Gaul <no...@github.com> on 2013/12/16 20:39:00 UTC

[jclouds] JCLOUDS-410. Deprecate Payload.getInput (#235)

We plan to transition Payload to ByteSource in the next major release.
Unfortunately Payload.getInput mask its checked exception and
ByteSource.getInput is final so we cannot continue to mask them.
Deprecation of getInput and addition openStream allows us to
transition callers from the former to the latter.
You can merge this Pull Request by running:

  git pull https://github.com/maginatics/jclouds open-stream

Or you can view, comment on it, or merge it online at:

  https://github.com/jclouds/jclouds/pull/235

-- Commit Summary --

  * JCLOUDS-410. Deprecate Payload.getInput

-- File Changes --

    M core/src/main/java/org/jclouds/io/Payload.java (9)
    M core/src/main/java/org/jclouds/io/payloads/BaseCipherPayload.java (2)
    M core/src/main/java/org/jclouds/io/payloads/BasePayload.java (11)
    M core/src/main/java/org/jclouds/io/payloads/ByteArrayPayload.java (2)
    M core/src/main/java/org/jclouds/io/payloads/DelegatingPayload.java (8)
    M core/src/main/java/org/jclouds/io/payloads/FilePayload.java (11)
    M core/src/main/java/org/jclouds/io/payloads/InputStreamPayload.java (2)
    M core/src/main/java/org/jclouds/io/payloads/InputStreamSupplierPayload.java (13)
    M core/src/main/java/org/jclouds/io/payloads/MultipartForm.java (10)
    M core/src/main/java/org/jclouds/io/payloads/PhantomPayload.java (2)
    M core/src/main/java/org/jclouds/io/payloads/StreamingPayload.java (9)
    M core/src/main/java/org/jclouds/io/payloads/StringPayload.java (2)
    M core/src/main/java/org/jclouds/io/payloads/UrlEncodedFormPayload.java (2)
    M drivers/apachehc/src/main/java/org/jclouds/http/apachehc/ApacheHCUtils.java (7)

-- Patch Links --

https://github.com/jclouds/jclouds/pull/235.patch
https://github.com/jclouds/jclouds/pull/235.diff

Re: [jclouds] JCLOUDS-410. Deprecate Payload.getInput (#235)

Posted by Andrew Gaul <no...@github.com>.
Spurious ```ConcurrentOpenSocketFinderTest.testChecksSocketsConcurrently``` failure.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/235#issuecomment-30699733

Re: [jclouds] JCLOUDS-410. Deprecate Payload.getInput (#235)

Posted by Andrew Phillips <no...@github.com>.
Pending PR builders +1, looks good to me. Thanks, @andrewgaul!

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/235#issuecomment-30693556

Re: [jclouds] JCLOUDS-410. Deprecate Payload.getInput (#235)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #477](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/477/) SUCCESS
This pull request looks good

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/235#issuecomment-30697559

Re: [jclouds] JCLOUDS-410. Deprecate Payload.getInput (#235)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-java-7-pull-requests #940](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/940/) UNSTABLE
Looks like there's a problem with this pull request

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/235#issuecomment-30697541