API refactoring: Should WARecord really be an inner class of RecordTransformers?

lintool / warcbase

Warcbase is an open-source platform for managing analyzing web archives

http://warcbase.org/

161 stars 47 forks source link

Closed lintool closed 8 years ago

lintool commented 8 years ago

I'm wondering why WARecord is an inner class of RecordTransformers?

Shouldn't we actually have something like:

Note they shouldn't actually be in the matchbox package since they aren't UDFs, no? Also, ArchiveRecord instead of WARecord to clearer?

lintool commented 8 years ago

Fixed commit ba2b44c52b35cb026c5665f250d77f4c9b586a80