Re: [jclouds] Remove StreamingPayload (#297)

2014-02-19 Thread Andrew Gaul 
I believe ```StreamingPayload``` and ```WriteTo``` predate Guava alternatives 
like ```ByteSource```.

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

Re: [jclouds] Remove StreamingPayload (#297)

2014-02-19 Thread Andrew Phillips 
Code changes themselves look fine (pending pull request builders...wonder what 
they are doing?). 

Reason for this? Cleanup? Deprecation? Risk reduction?

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

Re: [jclouds] Remove StreamingPayload (#297)

2014-02-19 Thread CloudBees pull request builder plugin 
[jclouds-java-7-pull-requests 
#1081](https://jclouds.ci.cloudbees.com/job/jclouds-java-7-pull-requests/1081/) 
SUCCESS
This pull request looks good

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

Re: [jclouds] Remove StreamingPayload (#297)

2014-02-19 Thread BuildHive 
[jclouds ยป jclouds 
#855](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/855/) SUCCESS
This pull request looks good
[(what's this?)](https://www.cloudbees.com/what-is-buildhive)

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

Re: [jclouds] Remove StreamingPayload (#297)

2014-02-19 Thread Andrew Phillips 
 Generally this commit is part of my effort to move jclouds to modern Guava 
 practices

Thanks for explaining, @andrewgaul! +1 from me.

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

Re: [jclouds] Remove StreamingPayload (#297)

2014-02-19 Thread CloudBees pull request builder plugin 
[jclouds-pull-requests 
#610](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/610/) SUCCESS
This pull request looks good

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

Re: [jclouds] Remove StreamingPayload (#297)

2014-02-19 Thread Andrew Gaul 
Pushed to master.

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

Re: [jclouds] Remove StreamingPayload (#297)

2014-02-19 Thread Andrew Gaul 
Closed #297.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/297