You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by Adrian Cole <no...@github.com> on 2014/10/05 22:58:48 UTC

[jclouds] remove deprecated executor accessors (#549)

https://issues.apache.org/jira/browse/JCLOUDS-153
You can merge this Pull Request by running:

  git pull https://github.com/adriancole/jclouds adrian.remove-deprecated-executor-accessors

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

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

-- Commit Summary --

  * JCLOUDS-153 Remove deprecated ExecutorService accessors.

-- File Changes --

    M compute/src/main/java/org/jclouds/compute/internal/UtilsImpl.java (14)
    M compute/src/test/java/org/jclouds/compute/internal/BaseComputeServiceLiveTest.java (8)
    M core/src/main/java/org/jclouds/rest/Utils.java (15)
    M core/src/main/java/org/jclouds/rest/internal/UtilsImpl.java (29)

-- Patch Links --

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

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

Re: [jclouds] Remove deprecated ExecutorService and HttpAsyncClient accessors (#549)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #1228](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1228/) 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/549#issuecomment-57954280

Re: [jclouds] Remove deprecated ExecutorService and HttpAsyncClient accessors (#549)

Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #1715](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1715/) FAILURE
Looks like there's a problem with this pull request
[(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/549#issuecomment-57972492

Re: [jclouds] Remove deprecated ExecutorService and HttpAsyncClient accessors (#549)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #1231](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1231/) 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/549#issuecomment-57956050

Re: [jclouds] Remove deprecated ExecutorService and HttpAsyncClient accessors (#549)

Posted by BuildHive <no...@github.com>.
[jclouds » jclouds #1717](https://buildhive.cloudbees.com/job/jclouds/job/jclouds/1717/) ABORTED

[(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/549#issuecomment-57972797

Re: [jclouds] Remove deprecated ExecutorService and HttpAsyncClient accessors (#549)

Posted by Adrian Cole <no...@github.com>.
added checkstyle fix.

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

Re: [jclouds] Remove deprecated ExecutorService and HttpAsyncClient accessors (#549)

Posted by Adrian Cole <no...@github.com>.
cherry-picked into master, 1.8.x

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

Re: [jclouds] remove deprecated executor accessors (#549)

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

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

Re: [jclouds] Remove deprecated ExecutorService and HttpAsyncClient accessors (#549)

Posted by Adrian Cole <no...@github.com>.
@demobox sure.. weird that I don't get these failures on my workstation.. will have a look.

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

Re: [jclouds] Remove deprecated ExecutorService and HttpAsyncClient accessors (#549)

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

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

Re: [jclouds] Remove deprecated ExecutorService and HttpAsyncClient accessors (#549)

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

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

Re: [jclouds] Remove deprecated ExecutorService and HttpAsyncClient accessors (#549)

Posted by Andrew Phillips <no...@github.com>.
> @@ -56,34 +55,12 @@
>  
>     Json json();
>  
> -   /**
> -    * 
> -    * @deprecated will be removed in jclouds 1.7, as async interfaces are no

Wow, this one hung on for quite a while, eh? ;-)

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/549/files#r18438119

Re: [jclouds] Remove deprecated ExecutorService and HttpAsyncClient accessors (#549)

Posted by CloudBees pull request builder plugin <no...@github.com>.
[jclouds-pull-requests #1232](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1232/) 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/549#issuecomment-57957004

Re: [jclouds] Remove deprecated ExecutorService and HttpAsyncClient accessors (#549)

Posted by Andrew Phillips <no...@github.com>.
> jclouds-pull-requests #1228 UNSTABLE

A bunch of [Checkstyle violations](https://jclouds.ci.cloudbees.com/job/jclouds-pull-requests/1228/violations/) here. Could you have a look at those, @adriancole? Thanks!

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