forked from flyteorg/flyte
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Completable Futures utility (flyteorg#65)
- Loading branch information
Ketan Umare
authored
Jun 17, 2020
1 parent
1b51b7e
commit 223ba85
Showing
2 changed files
with
201 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,116 @@ | ||
// This module implements a simple Async Futures for golang | ||
// Usage: | ||
// f := NewAsyncFuture(childCtx, func(ctx2 context.Context) (interface{}, error) { | ||
// can do large async / non blocking work | ||
// return ... | ||
// } | ||
// f.Ready() // can be checked for completion | ||
// f.Get() .. will block till the given sub-routine returns | ||
package data | ||
|
||
import ( | ||
"context" | ||
"fmt" | ||
"sync" | ||
) | ||
|
||
// Provides a Future API for asynchronous completion of tasks | ||
type Future interface { | ||
// Returns true if the Future is ready and either a value or error is available. Once Ready returns True, Get should return immediately | ||
Ready() bool | ||
// Get is a potentially blocking call, that returns the asynchronously computed value or an error | ||
// If Get is called before Ready() returns True, then it will block till the future has been completed | ||
Get(ctx context.Context) (interface{}, error) | ||
} | ||
|
||
// This is a synchronous future, where the values are available immediately on construction. This is used to maintain a synonymous API with both | ||
// Async and Sync tasks | ||
type SyncFuture struct { | ||
// The actual value | ||
val interface{} | ||
// OR an error | ||
err error | ||
} | ||
|
||
// Always returns true | ||
func (s SyncFuture) Ready() bool { | ||
return true | ||
} | ||
|
||
// returns the previously provided value / error | ||
func (s *SyncFuture) Get(_ context.Context) (interface{}, error) { | ||
return s.val, s.err | ||
} | ||
|
||
// Creates a new "completed" future that matches the async computation api | ||
func NewSyncFuture(val interface{}, err error) *SyncFuture { | ||
return &SyncFuture{ | ||
val: val, | ||
err: err, | ||
} | ||
} | ||
|
||
// ErrAsyncFutureCanceled is returned when the async future is cancelled by invoking the cancel function on the context | ||
var ErrAsyncFutureCanceled = fmt.Errorf("async future was canceled") | ||
|
||
// An asynchronously completing future | ||
type AsyncFuture struct { | ||
sync.Mutex | ||
doneChannel chan bool | ||
cancelFn context.CancelFunc | ||
// The actual value | ||
val interface{} | ||
// Or an error | ||
err error | ||
ready bool | ||
} | ||
|
||
func (f *AsyncFuture) set(val interface{}, err error) { | ||
f.Lock() | ||
defer f.Unlock() | ||
f.val = val | ||
f.err = err | ||
f.ready = true | ||
f.doneChannel <- true | ||
} | ||
|
||
func (f *AsyncFuture) get() (interface{}, error) { | ||
f.Lock() | ||
defer f.Unlock() | ||
return f.val, f.err | ||
} | ||
|
||
// returns whether the future is completed | ||
func (f *AsyncFuture) Ready() bool { | ||
f.Lock() | ||
defer f.Unlock() | ||
return f.ready | ||
} | ||
|
||
// Returns results (interface{} or an error) OR blocks till the results are available. | ||
// If context is cancelled while waiting for results, an ErrAsyncFutureCanceled is returned | ||
func (f *AsyncFuture) Get(ctx context.Context) (interface{}, error) { | ||
select { | ||
case <-ctx.Done(): | ||
f.cancelFn() | ||
return nil, ErrAsyncFutureCanceled | ||
case <-f.doneChannel: | ||
return f.get() | ||
} | ||
} | ||
|
||
// Creates a new Async future, that will call the method `closure` and return the results from the execution of | ||
// this method | ||
func NewAsyncFuture(ctx context.Context, closure func(context.Context) (interface{}, error)) *AsyncFuture { | ||
childCtx, cancel := context.WithCancel(ctx) | ||
f := &AsyncFuture{ | ||
doneChannel: make(chan bool, 1), | ||
cancelFn: cancel, | ||
} | ||
|
||
go func(ctx2 context.Context, fut *AsyncFuture) { | ||
val, err := closure(ctx2) | ||
fut.set(val, err) | ||
}(childCtx, f) | ||
return f | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,85 @@ | ||
package data | ||
|
||
import ( | ||
"context" | ||
"fmt" | ||
"runtime" | ||
"testing" | ||
"time" | ||
|
||
"github.com/stretchr/testify/assert" | ||
) | ||
|
||
func TestNewSyncFuture(t *testing.T) { | ||
type args struct { | ||
val interface{} | ||
err error | ||
} | ||
tests := []struct { | ||
name string | ||
args args | ||
}{ | ||
{"val", args{val: "val"}}, | ||
{"nil-val", args{}}, | ||
{"error", args{err: fmt.Errorf("err")}}, | ||
} | ||
for _, tt := range tests { | ||
t.Run(tt.name, func(t *testing.T) { | ||
got := NewSyncFuture(tt.args.val, tt.args.err) | ||
assert.NotNil(t, got) | ||
assert.Equal(t, tt.args.val, got.val) | ||
assert.Equal(t, tt.args.err, got.err) | ||
assert.True(t, got.Ready()) | ||
v, err := got.Get(context.TODO()) | ||
assert.Equal(t, tt.args.val, v) | ||
assert.Equal(t, tt.args.err, err) | ||
}) | ||
} | ||
} | ||
|
||
func TestAsyncFuture(t *testing.T) { | ||
|
||
const val = "val" | ||
t.Run("immediate-return-val", func(t *testing.T) { | ||
v := val | ||
err := fmt.Errorf("err") | ||
af := NewAsyncFuture(context.TODO(), func(ctx context.Context) (interface{}, error) { | ||
return v, err | ||
}) | ||
assert.NotNil(t, af) | ||
rv, rerr := af.Get(context.TODO()) | ||
assert.Equal(t, v, rv) | ||
assert.Equal(t, err, rerr) | ||
assert.True(t, af.Ready()) | ||
}) | ||
|
||
t.Run("wait-return-val", func(t *testing.T) { | ||
v := val | ||
err := fmt.Errorf("err") | ||
af := NewAsyncFuture(context.TODO(), func(ctx context.Context) (interface{}, error) { | ||
time.Sleep(time.Second * 1) | ||
return v, err | ||
}) | ||
runtime.Gosched() | ||
assert.NotNil(t, af) | ||
rv, rerr := af.Get(context.TODO()) | ||
assert.Equal(t, v, rv) | ||
assert.Equal(t, err, rerr) | ||
assert.True(t, af.Ready()) | ||
}) | ||
|
||
t.Run("timeout", func(t *testing.T) { | ||
v := val | ||
ctx := context.TODO() | ||
af := NewAsyncFuture(ctx, func(ctx context.Context) (interface{}, error) { | ||
time.Sleep(time.Second * 5) | ||
return v, nil | ||
}) | ||
runtime.Gosched() | ||
cctx, cancel := context.WithCancel(ctx) | ||
cancel() | ||
_, rerr := af.Get(cctx) | ||
assert.Error(t, rerr) | ||
assert.Equal(t, ErrAsyncFutureCanceled, rerr) | ||
}) | ||
} |