search

twitter-archive / commons

Twitter common libraries for python and the JVM (deprecated)
http://twitter.github.com/commons
Other
2.1k stars 565 forks source link

fixed issue #329 #456

Closed AmberSahdev closed 4 years ago

AmberSahdev commented 6 years ago

Problem

This fixes issue #329 , where one process was reading a file partially written by another process.

Solution

Only open the file once the 'if onwait' is true. This is done by moving the file read operation within the if statement.

CLAassistant commented 6 years ago

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

grimreaper commented 4 years ago

Thank you for your contribution. Unfortunately, we’re not going to continue maintaining twitter-commons and are archiving all pull requests.