search

ddavisqa / google-refine

Automatically exported from code.google.com/p/google-refine
0 stars 0 forks source link

SeparatorRowParser handles blanks differently from TsvCsvRowParser #43

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Using r793 from SVN.
Import a CSV file with a blank data point e.g. "value1","value2",,"value4"

SeparatorRowParser adds the empty cell to the model. (row.cells.getSize() == 
4)
TsvCsvRowParser omits the empty cell from the model.  (row.cells.getSize() == 
3)

What's the expected behaviour?  Should it be added or omitted?

Original issue reported on code.google.com by iainsproat on 17 May 2010 at 6:57

GoogleCodeExporter commented 8 years ago 
Should be added, but with as a null.  row.cells.getSize() == 4;  
row.cells.get(2) == 
null;

Fixed in r797.

Original comment by iainsproat on 17 May 2010 at 12:00

GoogleCodeExporter commented 8 years ago

Original comment by tfmorris on 18 Sep 2012 at 2:21

GoogleCodeExporter commented 8 years ago

Original comment by tfmorris on 18 Sep 2012 at 2:56