search

palantir / atlasdb

Transactional Distributed Database Layer
https://palantir.github.io/atlasdb/
Apache License 2.0
50 stars 9 forks source link

Jackson Persister accepts null values #3238

Open fscreene opened 6 years ago

fscreene commented 6 years ago

We're using JacksonPersister to serialize ResourceIdentifiers and ran into an issue that was tough to debug. If you use the Persister with values that are actually null, it seems they'll be serialized as the string "null". While this can obviously be avoided by not putting null values into rows in the first place, it would be nice if this was inherently disallowed, as it can make for very difficult debugging if, for example, you expect those values to be not-set but instead they're set to "null".

tboam commented 6 years ago

We don't have the time to work on this right now, moving to the backlog.