Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: replace unbuffered channels with buffered channels #1668

Merged
merged 14 commits into from
Jul 9, 2021
Merged

chore: replace unbuffered channels with buffered channels #1668

Show file tree
Hide file tree
Changes from 13 commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
6885bae
add buffer to channels
edwardmack Jul 1, 2021
797b0bf
add buffers to channels in tests
edwardmack Jul 1, 2021
0bbd004
Merge branch 'development' into ed/add_channel_buffers
edwardmack Jul 2, 2021
c84976a
remove buffers from channels that shouldn't be buffered
edwardmack Jul 2, 2021
634050b
Merge branch 'development' into ed/add_channel_buffers
edwardmack Jul 2, 2021
8ed5ee6
Merge branch 'development' into ed/add_channel_buffers
edwardmack Jul 6, 2021
39eb00f
added DEFAULT_BUFFERS_SIZE const
edwardmack Jul 6, 2021
504e4f6
Merge branch 'development' into ed/add_channel_buffers
edwardmack Jul 7, 2021
6395b5c
lint
edwardmack Jul 7, 2021
79db38b
Merge branch 'development' into ed/add_channel_buffers
edwardmack Jul 7, 2021
c8936e5
Merge branch 'development' into ed/add_channel_buffers
edwardmack Jul 7, 2021
3e43488
addres comments
edwardmack Jul 8, 2021
5cd262c
Merge branch 'development' into ed/add_channel_buffers
edwardmack Jul 8, 2021
651e07a
Merge branch 'development' into ed/add_channel_buffers
edwardmack Jul 9, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions dot/rpc/subscription/listeners_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -121,8 +121,8 @@ func TestBlockFinalizedListener_Listen(t *testing.T) {
}

func TestExtrinsicSubmitListener_Listen(t *testing.T) {
notifyImportedChan := make(chan *types.Block)
notifyFinalizedChan := make(chan *types.FinalisationInfo)
notifyImportedChan := make(chan *types.Block, 100)
notifyFinalizedChan := make(chan *types.FinalisationInfo, 100)

mockConnection := &MockWSConnAPI{}
esl := ExtrinsicSubmitListener{
Expand Down
11 changes: 7 additions & 4 deletions dot/rpc/subscription/websocket.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,9 @@ var errCannotReadFromWebsocket = errors.New("cannot read message from websocket"
var errCannotUnmarshalMessage = errors.New("cannot unmarshal webasocket message data")
var logger = log.New("pkg", "rpc/subscription")

// DEFAULT_BUFFER_SIZE buffer size for channels
const DEFAULT_BUFFER_SIZE = 100

// WSConn struct to hold WebSocket Connection references
type WSConn struct {
Wsconn *websocket.Conn
Expand Down Expand Up @@ -228,7 +231,7 @@ func (c *WSConn) unsubscribeStorageListener(reqID float64, l Listener, _ interfa

func (c *WSConn) initBlockListener(reqID float64, _ interface{}) (Listener, error) {
bl := &BlockListener{
Channel: make(chan *types.Block),
Channel: make(chan *types.Block, DEFAULT_BUFFER_SIZE),
wsconn: c,
}

Expand Down Expand Up @@ -260,7 +263,7 @@ func (c *WSConn) initBlockListener(reqID float64, _ interface{}) (Listener, erro

func (c *WSConn) initBlockFinalizedListener(reqID float64, _ interface{}) (Listener, error) {
bfl := &BlockFinalizedListener{
channel: make(chan *types.FinalisationInfo),
channel: make(chan *types.FinalisationInfo, DEFAULT_BUFFER_SIZE),
wsconn: c,
}

Expand Down Expand Up @@ -299,10 +302,10 @@ func (c *WSConn) initExtrinsicWatch(reqID float64, params interface{}) (Listener

// listen for built blocks
esl := &ExtrinsicSubmitListener{
importedChan: make(chan *types.Block),
importedChan: make(chan *types.Block, DEFAULT_BUFFER_SIZE),
wsconn: c,
extrinsic: types.Extrinsic(extBytes),
finalisedChan: make(chan *types.FinalisationInfo),
finalisedChan: make(chan *types.FinalisationInfo, DEFAULT_BUFFER_SIZE),
}

if c.BlockAPI == nil {
Expand Down