snowplow/snowplow — pull request #60 by mtibben (2012-10-19 22:47:27+00:00)
Title: Ensure exceptions are not thrown

Hey @alexanderdean,

This change is to do exception handling within the serde

It's is a reworked version of what we were discussing at https://github.com/99designs/snowplow/pull/1

I've taken the approach suggested by @yalisassoon in that we capture all the lines, even if data is missing.

I've added one config parameter:
```
snowplow.serde.ignore_errors = 1
```
which will cause the serde to return null instead of throwing an exception

This does break a few tests, as the behaviour is changed when bad data is present - it will just always null out the field instead of returning a nulled object. So this should also address https://github.com/snowplow/snowplow/issues/52

If everyone is happy with this approach I'll fix the rest of the tests.

Any comments and feedback appreciated!

cc @larsyencken