chenejac / VIVOTestMigrationJIRANotClosed

0 stars 0 forks source link

VIVO-1840: Tab-separated files are not read properly by Convert CSV to RDF ingest tool #68

Open chenejac opened 4 years ago

chenejac commented 4 years ago

Brian Lowe (Migrated from VIVO-1840) said:

Selecting  the "tab separated" radio button in the Convert CSV to RDF ingest tool causes the text from all cells in each row to run together in a single string.  The culprit appears to be near line 70 of Csv2Rdf, where the method .withRecordSeparator() should really be .withDelimiter():

CSVParser cReader = new CSVParser(new InputStreamReader(fis),

                           CSVFormat.DEFAULT.withDelimiter(separatorChar)

                                                     .withQuote(quoteChar));

 

Instead of

 

CSVParser cReader = new CSVParser(new InputStreamReader(fis),

                           CSVFormat.DEFAULT.withRecordSeparator(separatorChar)

                                                     .withQuote(quoteChar));

 

When we fix this we should also add a unit test.

chenejac commented 4 years ago

Andrew Woods said:

See vivo-tech email thread: https://groups.google.com/d/msg/vivo-tech/dACPgkqesjE/4gOBGsTjCQAJ