Specify is it source or sink is absent in mocked sources/sinks

twitter / scalding

A Scala API for Cascading

http://twitter.com/scalding

Apache License 2.0

3.5k stars 706 forks source link

Specify is it source or sink is absent in mocked sources/sinks #1866

Closed ttim closed 6 years ago

ttim commented 6 years ago

Recently couple of engineers in Twitter were confused by error message scalding prints in case if test source or sink isn't properly mocked in JobTest.

In this PR I made this message more specific and print either source or sink based on access mode used for this source's tap.

CLAassistant commented 6 years ago

Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}