[BOOKKEEPER-837] UpdateLedgerOp - Replace AbstractFuture with SettableFuture

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 4.4.0, 4.3.1
• Component/s: bookkeeper-auto-recovery, bookkeeper-client, bookkeeper-server
• Assignee: Rakesh Radhakrishnan
• Reporter: Rakesh Radhakrishnan
• Created: 06/Feb/15 15:16
• Updated: 16/May/16 21:47
• Resolved: 12/Mar/15 07:37

Description

Review comment : Using AbstractFuture is too tight with guava's internal implementation. Instead it would be good to use a variable for the settable future.

 class Read... implements GenericCallback {

 SettableFuture future = ...;
  public void operationComplete(..) {
    future.set(null); 
    // future.setException(..)
  }
}