client could be part of a pool and get reused for other transfers (common case on the source side of transfer).
it's the actual flow that should be interrupted eventually. this PR doesn't take care of this, it's just patching the bug in the simplest way possible.
Further notes
List other areas of code that have changed but are not necessarily linked to the main feature. This could be method
signature changes, package declarations, bugs that were encountered and were fixed inline, etc.
What this PR changes/adds
Stop closing the client when
DataSource
is closedWhy it does that
client could be part of a pool and get reused for other transfers (common case on the source side of transfer). it's the actual flow that should be interrupted eventually. this PR doesn't take care of this, it's just patching the bug in the simplest way possible.
Further notes
List other areas of code that have changed but are not necessarily linked to the main feature. This could be method signature changes, package declarations, bugs that were encountered and were fixed inline, etc.
Linked Issue(s)
Closes #455
_Please be sure to take a look at the contributing guidelines and our etiquette for pull requests._