[GitHub] [metamodel] kaspersorensen commented on issue #221: Access CSVParser in a synchronized manner

[GitBox <gi...@apache.org>]

kaspersorensen commented on issue #221: Access CSVParser in a synchronized manner
URL: https://github.com/apache/metamodel/pull/221#issuecomment-501982545
 
 
   That's a great find and we need to fix it and certainly keep the test case. But I wonder if the solution is right. We're going to be adding a ton of locking and unlocking to our parallel reader which might very well just make it slower than the non-parallel one. I think a "shared nothing" approach to the problem might be better. Could we either instantiate the CsvParser every time, or is there a static way of parsing a line which does not interfere with the CsvParser's state?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services