Update replication.py

[gaetan-h]

thanks for finding the bug
dst_ts was not declared in the thread function
i have declared it now and it should be passed to the copy function
Please test (with low batch size) and let me know

[cognite-cicd]

Checkmarx scan completed with the following findings

Lines	Severity	Category	File	Link

[jenkins-cognite]

[pr-bot]
Run this build with docker run --rm -it cognite/cognite-replicator-dev:pr-123

[codecov]

Codecov Report

Merging #123 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #123   +/-   ##
======================================
  Coverage    42.4%   42.4%           
======================================
  Files          10      10           
  Lines         724     724           
======================================
  Hits          307     307           
  Misses        417     417

Impacted Files	Coverage Δ
cognite/replicator/replication.py	76.56% <ø> (ø)	⬆️

[copperwire]

Tested with a batch size of 100, works fine.

[gaetan-h]

Great, thanks for the feedback

…

On Wed, 26 Feb 2020, 15:58 Robert Solli, ***@***.***> wrote: Tested with a batch size of 100, works fine. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#123?email_source=notifications&email_token=AOHWHMXTIFCFCJPC3WD4273REZ7R5A5CNFSM4K3NVMPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENARTPI#issuecomment-591468989>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AOHWHMXITXMXKRD6YAMQDL3REZ7R5ANCNFSM4K3NVMPA> .