[
https://issues.apache.org/jira/browse/BEAM-3607?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16858964#comment-16858964
]
Kenneth Knowles commented on BEAM-3607:
---------------------------------------
Probably. I don't think there's any good reason to use {{checkNotNull}} as
Guava provides it.
> Move checkNotNull arg checks to a new checkArgumentNotNull
> ----------------------------------------------------------
>
> Key: BEAM-3607
> URL: https://issues.apache.org/jira/browse/BEAM-3607
> Project: Beam
> Issue Type: Sub-task
> Components: runner-core, sdk-java-core
> Reporter: Kenneth Knowles
> Priority: Major
> Labels: beginner, newbie, starter
>
> The simple fact is that {{checkNotNull}} throws NPE which to users looks like
> a core dump sort of failure. It throws a "500 error" when we usually intend a
> "400 error", so to speak, so the bugs get filed on the wrong components, or
> users don't know they passed the wrong thing, etc.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
