search

twitter / scalding

A Scala API for Cascading
http://twitter.com/scalding
Apache License 2.0
3.49k stars 703 forks source link

Add in new test and filter handling #1841

Closed ianoc closed 6 years ago

ianoc commented 6 years ago

Expand the handling to filters too

CLAassistant commented 6 years ago

CLA assistant check
All committers have signed the CLA.

ianoc commented 6 years ago

This test fails and code fixes it

johnynek commented 6 years ago

👍

I am worried there are more cases now, but it seems like this is an improvement.

we can merge and add more tests.