Menu

#329 CsvURLProducer swallows exception stack

v2.3.*
closed-fixed
Bug (232)
2.5.0
5
2014-04-20
2012-07-19
Art
No

This code

catch (CsvParserException e) {
throw new DataSetException("error parsing CSV for URL: '" + url + "'");

doesn't chain the exception so the stack trace is lost. Patch attached.

[Code review hat on] This class is copy/pasted from CsvProducer and ought to have the duplication refactored out

Discussion

  • Art

    Art - 2012-07-19
     
  • Jeff Jensen

    Jeff Jensen - 2012-09-30

    Thanks for reporting and the fix.

    Commit 1256.

     
  • Jeff Jensen

    Jeff Jensen - 2012-09-30
    • assigned_to: gommma --> jeffjensen
    • status: open --> closed-fixed
     
  • Jeff Jensen

    Jeff Jensen - 2014-04-20
    • Fixed Release: --> 2.5.0
    • Found Release: --> v2.3.*
     

Log in to post a comment.