Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fix NPE in DoubleSerializer for nonexistent columns #421

Merged
merged 1 commit into from Apr 5, 2012

Conversation

Projects
None yet
2 participants
Contributor

amrik commented Feb 17, 2012

DoubleSerializer uses LongSerializer internally. However whenever LongSerializer returns null (such as when the column does not exist) this would cause a NullPointerException.

Here I add some guards similar to what already exist in LongSerializer.

Collaborator

zznate commented Apr 5, 2012

Good catch - thanks.

@zznate zznate pushed a commit that referenced this pull request Apr 5, 2012

Nate McCall Merge pull request #421 from amrik/master
Fix NPE in DoubleSerializer for nonexistent columns
59de8d1

@zznate zznate merged commit 59de8d1 into hector-client:master Apr 5, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment