[ 
https://issues.apache.org/jira/browse/HBASE-25794?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Peter Somogyi updated HBASE-25794:
----------------------------------
    Fix Version/s: hbase-operator-tools-1.2.0
       Resolution: Fixed
           Status: Resolved  (was: Patch Available)

Merged to master. Thanks [~sk...@cloudera.com]!

> Fix checkstyle violations in hbase-table-reporter module
> --------------------------------------------------------
>
>                 Key: HBASE-25794
>                 URL: https://issues.apache.org/jira/browse/HBASE-25794
>             Project: HBase
>          Issue Type: Task
>          Components: hbase-operator-tools
>            Reporter: Peter Somogyi
>            Assignee: Shanmukha Haripriya Kota
>            Priority: Trivial
>              Labels: beginner
>             Fix For: hbase-operator-tools-1.2.0
>
>
> The hbase-table-reporter module in hbase-operator-tools has a bunch of 
> checkstyle violations.
> {noformat}
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[22] 
> (imports) ImportOrder: Wrong order for 'org.apache.commons.cli.Option' import.
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[23] 
> (imports) ImportOrder: Wrong order for 
> 'org.apache.commons.cli.CommandLineParser' import.
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[24] 
> (imports) ImportOrder: Wrong order for 'org.apache.commons.cli.CommandLine' 
> import.
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[34,8] 
> (imports) UnusedImports: Unused import - org.apache.hadoop.hbase.HRegionInfo.
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[37] 
> (imports) AvoidStarImport: Using the '.*' form of import should be avoided - 
> org.apache.hadoop.hbase.client.*.
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[41] 
> (imports) ImportOrder: Wrong order for 'java.io.BufferedWriter' import.
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[66,1] 
> (design) HideUtilityClassConstructor: Utility classes should not have a 
> public or default constructor.
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[94] 
> (sizes) LineLength: Line is longer than 100 characters (found 120).
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[126] 
> (sizes) LineLength: Line is longer than 100 characters (found 105).
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[132] 
> (sizes) LineLength: Line is longer than 100 characters (found 106).
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[247] 
> (indentation) Indentation: 'throws' has incorrect indentation level 2, 
> expected level should be 4.
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[275] 
> (sizes) LineLength: Line is longer than 100 characters (found 112).
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[279] 
> (sizes) LineLength: Line is longer than 100 characters (found 105).
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[308] 
> (sizes) LineLength: Line is longer than 100 characters (found 101).
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[370] 
> (sizes) LineLength: Line is longer than 100 characters (found 121).
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[371] 
> (sizes) LineLength: Line is longer than 100 characters (found 108).
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[375] 
> (sizes) LineLength: Line is longer than 100 characters (found 102).
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[401] 
> (sizes) LineLength: Line is longer than 100 characters (found 113).
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[407] 
> (sizes) LineLength: Line is longer than 100 characters (found 112).
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[410] 
> (sizes) LineLength: Line is longer than 100 characters (found 107).
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[437] 
> (sizes) LineLength: Line is longer than 100 characters (found 114).
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[438] 
> (sizes) LineLength: Line is longer than 100 characters (found 109).
> [ERROR] src/main/java/org/apache/hbase/reporter/TableReporter.java:[445] 
> (indentation) Indentation: 'throws' has incorrect indentation level 2, 
> expected level should be 4.
> [ERROR] src/test/java/org/apache/hbase/reporter/TestTableReporter.java:[27] 
> (imports) ImportOrder: Wrong order for 'java.util.ArrayList' import.
> [ERROR] src/test/java/org/apache/hbase/reporter/TestTableReporter.java:[30] 
> (imports) ImportOrder: Wrong order for 
> 'org.apache.hadoop.hbase.shaded.junit.framework.TestCase.assertEquals' import.
> [ERROR] src/test/java/org/apache/hbase/reporter/TestTableReporter.java:[59] 
> (sizes) LineLength: Line is longer than 100 characters (found 110).
> [ERROR] src/test/java/org/apache/hbase/reporter/TestTableReporter.java:[75] 
> (sizes) LineLength: Line is longer than 100 characters (found 110).
> [ERROR] src/test/java/org/apache/hbase/reporter/TestTableReporter.java:[80] 
> (sizes) LineLength: Line is longer than 100 characters (found 111).
> {noformat}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to