search

mperdeck / LINQtoCSV

Popular, easy to use library to read and write CSV files.
197 stars 112 forks source link

Added tests for parallel linq and DataRow. #1

Closed cofiem closed 10 years ago

cofiem commented 10 years ago

The test for DataRowAssertable fails. There are no DataRowItems in the DataRowAssertable objects, even though when debugging the yield return obj line (in ReadData<T> method in CsvContext.cs) does have DataRowItems.

mperdeck commented 10 years ago

Thanks cofiem for your contribution. I saw your pull request, which contains a unit test to test whether LINQ to CSV support PLINQ. And you found that currently it doesn't (the test fails).

When I developed LINQ to CSV, I didn't see the need to support PLINQ, but maybe I was wrong. If you want to add support for PLINQ to LINQ to CSV, please feel free to go ahead with this. Please keep your changes to a minimum to minimize chances of introducing a bug.

mperdeck commented 10 years ago

Closed due to no activity.