Andrew Crum wrote:

Oh, yeah, I forgot to mention that these two lines are redundant:


> $dbh->{csv_tables}->{table}->{skip_rows} = 0;
> $dbh->{csv_tables}->{table}->{col_names} = [qw( col1 col2 col3 col4)];

csv_skip_rows is automatically set to 0 if you specify the col_names 
parameter so you don't need to explicity set the skip.

Oh, and as long as we're at it, for portability, I'd advise not using 
the SQL reserved keyword "table" as the name of a table, it's allowed 
now in DBD::CSV for historical reasons, but definitely isn't allowed in 
some other  RDBMSs.

-- 
Jeff

Reply via email to