[
https://issues.apache.org/jira/browse/SQOOP-2795?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ze Jin updated SQOOP-2795:
--------------------------
Description:
The import in line 7 is never used. The field in line 13 is not used, and
methods in class SqoopJsonUtil are all static, so any intance of SqoopJsonUtil
is unnecessary.
According to Sonar rule squid:S1118 - Utility classes should not have public
constructors
You can find more information about the issue here:
https://dev.eclipse.org/sonar/coding_rules#q=squid:S1118
was:The import in line 7 is never used. The field in line 13 is not used, and
methods in class SqoopJsonUtil are all static, so any intance of SqoopJsonUtil
is unnecessary.
> [Polish code] Clean up useless code in class TestSqoopJsonUtil
> --------------------------------------------------------------
>
> Key: SQOOP-2795
> URL: https://issues.apache.org/jira/browse/SQOOP-2795
> Project: Sqoop
> Issue Type: Improvement
> Affects Versions: 1.4.6
> Reporter: Ze Jin
> Priority: Trivial
> Labels: newbie
> Fix For: 1.4.7
>
> Attachments: SQOOP-2795.0000.patch, SQOOP-2795.0001.patch
>
>
> The import in line 7 is never used. The field in line 13 is not used, and
> methods in class SqoopJsonUtil are all static, so any intance of
> SqoopJsonUtil is unnecessary.
> According to Sonar rule squid:S1118 - Utility classes should not have public
> constructors
> You can find more information about the issue here:
> https://dev.eclipse.org/sonar/coding_rules#q=squid:S1118
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
