You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by Ignasi Barrera <no...@github.com> on 2015/05/07 11:34:20 UTC

[jclouds-site] Minor addition to context definition (#166)

You can view, comment on, or merge this pull request online at:

  https://github.com/jclouds/jclouds-site/pull/166

-- Commit Summary --

  * Minor addition to context definition

-- File Changes --

    M start/concepts.md (4)

-- Patch Links --

https://github.com/jclouds/jclouds-site/pull/166.patch
https://github.com/jclouds/jclouds-site/pull/166.diff

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

Re: [jclouds-site] Minor addition to context definition (#166)

Posted by Andrew Phillips <no...@github.com>.
>  You can also get a view back from the ContextBuilder and _later_ unwrap it to access the underlying API or provider.
> +
> +The context also provides access to some low level resources, such as the dependency injection framework or the executors used to perform concurrent operations, so it is important to close it once you are done, to free all its associated resources.

Do we still need this para?

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

Re: [jclouds-site] Minor addition to context definition (#166)

Posted by Andrew Phillips <no...@github.com>.
> Pushed to master. Thanks!

Thanks, @nacx! See https://github.com/jclouds/jclouds-site/pull/167

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

Re: [jclouds-site] Minor addition to context definition (#166)

Posted by Ignasi Barrera <no...@github.com>.
Closed #166.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-site/pull/166#event-299263285

Re: [jclouds-site] Minor addition to context definition (#166)

Posted by Ignasi Barrera <no...@github.com>.
Pushed to [master](http://git-wip-us.apache.org/repos/asf/jclouds-site/commit/56a75bbb). Thanks!

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

Re: [jclouds-site] Minor addition to context definition (#166)

Posted by jclouds-commentator <no...@github.com>.
  Go to http://e3fc44bd4430654115f7-165e9c5ecddbeacd39e268108e8f046e.r46.cf5.rackcdn.com/ to review your changes.

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

Re: [jclouds-site] Minor addition to context definition (#166)

Posted by jclouds-commentator <no...@github.com>.
  Go to http://55de5352931516bd6330-55e1d72d54648c5093cb6da556c72f45.r69.cf5.rackcdn.com/ to review your changes.

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

Re: [jclouds-site] Minor addition to context definition (#166)

Posted by Ignasi Barrera <no...@github.com>.
Oh yes, I forgot to remove it. Thanks!

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

Re: [jclouds-site] Minor addition to context definition (#166)

Posted by Andrew Phillips <no...@github.com>.
> @@ -80,3 +80,7 @@ A context represents a specific connection to a particular provider. From the pe
>  Once you have created a context via the [ContextBuilder](http://javadocs.jclouds.cloudbees.net/org/jclouds/ContextBuilder.html) and are "connected" to a particular cloud service, you can either get any of the views that are supported by that provider, or go straight to the API or even to the provider level.

Suggestion: add it here? As follows:

"...provider level.

Creating a context is an expensive operation, so in general it is a good idea to create one context per credential and target cloud when the application starts and close it when it terminates. Contexts are thread-safe, so can be shared across your application.

It is important to [close a context](http://jclouds-javadocs.elasticbeanstalk.com/org/jclouds/Context.html) when you no longer needed, to free its associated resources."

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