Github user aledsage commented on a diff in the pull request:

    https://github.com/apache/brooklyn-server/pull/853#discussion_r142929325
  
    --- Diff: 
api/src/main/java/org/apache/brooklyn/api/mgmt/ExecutionContext.java ---
    @@ -86,18 +96,27 @@
         // TODO reference ImmediateSupplier when that class is moved to utils 
project
         @Beta
         <T> Maybe<T> getImmediately(Object callableOrSupplierOrTask);
    +    /** As {@link #getImmediately(Object)} but strongly typed for a task. 
*/
    +    // TODO deprecate
    +    @Beta
    +    <T> Maybe<T> getImmediately(TaskAdaptable<T> callableOrSupplierOrTask);
    +    /** As {@link #getImmediately(Object)} but strongly typed for a task 
factory. */
    +    @Beta
    +    <T> Maybe<T> getImmediately(TaskFactory<Task<T>> 
callableOrSupplierOrTask);
    --- End diff --
    
    Wrong parameter name.


---

Reply via email to