> @@ -66,8 +66,11 @@ protected void configure() {
>     }
>  
>     /**
> -    * Provides a cache for tokens. Cache is time based and expires after 59 
> minutes (the maximum time a token is
> -    * valid is 60 minutes)
> +    * Provides a cache for tokens. Cache is time based and by default 
> expires after 59 minutes 
> +    * (the maximum time a token is valid is 60 minutes).
> +    * This cache and expiry period is system-wide and does not attend to 
> per-instance expiry time
> +    * (e.g. "expires_in" from Google Compute -- which is set to the standard 
> 3600 seconds).
> +    * If per-instance expiry time is required, refactoring will be needed.

Odd comment for Javadoc...move the last line to an implementation comment?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-google/pull/14/files#r7587531

Reply via email to