search

nathanmarz / cascalog

Data processing on Hadoop without the hassle.
Other
1.38k stars 178 forks source link

Allow empty vector to be used as a generator. #229

Closed f355 closed 10 years ago

f355 commented 10 years ago

Fixes the issue that prevented [] to be used as a generator. Due to the fact that (seq []) returns nil, following exception was thrown:

IllegalArgumentException No implementation of method: :generator of protocol: #'cascalog.cascading.types/IGenerator found for class: nil
sritchie commented 10 years ago

Awesome. would you mind adding a test?

f355 commented 10 years ago

Sure, will add it a bit later.

f355 commented 10 years ago

The case is trivial, so I plunked it near other simple stuff. Maybe it needs to go elsewhere, though.

sritchie commented 10 years ago

looks good to me. Thanks for this