BUGS-NYU/schedge

ERROR: value too long for type character varying(6)

NicholasLYang opened this issue · 4 comments

We're using varchar(n) on certain fields, causing errors on scrape when the value is too big. We should get better reporting for this btw, it's unclear which course this is.

According to the Postgres wiki varchar(n) isn't advisable in the first place, so we should probably just change these to varchar unless there's a very good reason to the contrary.

Even if we're using varchar(n) as validation, we should probably catch this in code and spit out the row. That way we can tell if it's bad data or a genuine exception.

A1Liu commented

Are you on latest? I added error printing for this a few days ago.

A1Liu commented

Oh lol nvm I didn't push it to master so its not on your fork.

A1Liu commented

This has been fixed.