search

mauricioaniche / repodriller

a tool to support researchers on mining software repositories studies
174 stars 39 forks source link

Flush in the CSVWriter? #92

Open mauricioaniche opened 7 years ago

mauricioaniche commented 7 years ago

Do we still need it? It indeed has performance issues. See PR #90.

davisjam commented 7 years ago

Actually see PR #91

davisjam commented 7 years ago

To address this issue we need to evaluate the failure path to make sure close() is called on all PersistenceMethods. If so then we can safely remove this flush.