[
https://issues.apache.org/jira/browse/HADOOP-12514?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14977636#comment-14977636
]
Mingliang Liu commented on HADOOP-12514:
----------------------------------------
Hi [~andrew.wang], would you kindly review and commit the patch if it looks
good to your original comments in [HADOOP-12472]?
> Make static fields in GenericTestUtils for assertExceptionContains()
> package-private and final
> ----------------------------------------------------------------------------------------------
>
> Key: HADOOP-12514
> URL: https://issues.apache.org/jira/browse/HADOOP-12514
> Project: Hadoop Common
> Issue Type: Task
> Reporter: Mingliang Liu
> Assignee: Mingliang Liu
> Priority: Minor
> Attachments: HADOOP-12514.000.patch
>
>
> This is a follow up of [HADOOP-12472].
> It makes sense to make the following static fields package private instead of
> protected, as they are for test purpose and {{TestGenericTestUtils}} is in
> the same package as {{GenericTestUtils}}.
> - protected static String E_NULL_THROWABLE = "Null Throwable";
> - protected static String E_NULL_THROWABLE_STRING = "Null
> Throwable.toString() value";
> - protected static String E_UNEXPECTED_EXCEPTION = "but got unexpected
> exception";
> Meanwhile, we may need to make them final.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
