[
https://issues.apache.org/jira/browse/TEZ-4101?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17021581#comment-17021581
]
Jonathan Turner Eagles commented on TEZ-4101:
---------------------------------------------
[~abstractdog], there are so few checkState (throw IllegalStateException) and
checkArgument (IllegalArgumentException), so they could be done manually with a
if/throw block. If you want to create a class to encapsulate, that is fine
also. In that case, try to avoid creating two separate files that contain the
same logic.
> Eliminate some guava dependencies by Java8+ features - Preconditions
> --------------------------------------------------------------------
>
> Key: TEZ-4101
> URL: https://issues.apache.org/jira/browse/TEZ-4101
> Project: Apache Tez
> Issue Type: Improvement
> Reporter: László Bodor
> Assignee: László Bodor
> Priority: Major
> Attachments: TEZ-4101.01.patch, TEZ-4101.02.patch, TEZ-4101.03.patch
>
>
> 1. Preconditions.checkNotNull -> Objects.requireNonNull
> 2. wrap others into a tez common class
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
