search

ddavisqa / google-refine

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

Refactor importer API #112

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
The attached patch refactors the importer API so that:
 1. It uses separate interfaces for importers that support the Reader method and those that don't to allow them to be statically type checked.
 2. Uses a separate ImporterException to allow caller to distinguish between things that were handled (i/o exceptions, etc) and unexpected problems
 3. Adds a URL importer interface that can be used by importers that need neither a stream, nor a reader.

Please let me know if it needs revising in any way.

Original issue reported on code.google.com by tfmorris on 4 Aug 2010 at 8:10

Attachments:

GoogleCodeExporter commented 8 years ago 
Fixed by r1138

Original comment by dfhu...@google.com on 6 Aug 2010 at 6:15

GoogleCodeExporter commented 8 years ago

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