search

exasol / advanced-analytics-framework

Framework for building complex data analysis algorithms with Exasol
MIT License
0 stars 0 forks source link

#154: Move sending messages to peers to the BackgroundPeerState #156

Closed tkilias closed 1 year ago

tkilias commented 1 year ago

Fixes #154

tomuben commented 1 year ago

Based on the title of the PR I would have expected the send implementation now in BackgroundPeerState, but in fact it has been moved to BackgroundListenerInterface

tkilias commented 1 year ago

Based on the title of the PR I would have expected the send implementation now in BackgroundPeerState, but in fact it has been moved to BackgroundListenerInterface

You completely confused me, because I didn't move the receive in #157 completely to the BackgroundPeerState, but in fact the send went to the BackgroundPeerState.

see https://github.com/exasol/advanced-analytics-framework/pull/156/files#diff-f5de8b58b72b6a90f700cf884e203120ed3a1d8e4bbb666dda54c4d13ca9e02d