hbase-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Evgeny Ryabitskiy (JIRA)" <j...@apache.org>
Subject [jira] Issue Comment Edited: (HBASE-876) There are a large number of Java warnings in HBase
Date Sun, 11 Jan 2009 00:21:59 GMT

    [ https://issues.apache.org/jira/browse/HBASE-876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12662659#action_12662659
] 

apparition edited comment on HBASE-876 at 1/10/09 4:21 PM:
------------------------------------------------------------------

876_1.patch fixes all warnings in package:
org.apache.hadoop.hbase

      was (Author: apparition):
    First patch that fixes all warning in package:
org.apache.hadoop.hbase
  
> There are a large number of Java warnings in HBase
> --------------------------------------------------
>
>                 Key: HBASE-876
>                 URL: https://issues.apache.org/jira/browse/HBASE-876
>             Project: Hadoop HBase
>          Issue Type: Bug
>    Affects Versions: 0.18.0
>            Reporter: Jim Kellerman
>            Assignee: Evgeny Ryabitskiy
>            Priority: Minor
>             Fix For: 0.20.0
>
>         Attachments: 876_1.patch, 876_2.patch
>
>
> There are a large number of Java warnings in the current HBase code base including:
> - exceptions that do not define serialVersionUID
> - classes that use the raw type WritableComparable instead of WritableComparable<T>
> - classes or interfaces that declare public members that are not a part of the public
API. In this case they should be moved to a place where their visibility needs not be public.
Additionally, there are a number of classes that declare public members that need not be.
Make them protected or private or default as needed
> - methods that have unnecessary else clauses
> - potential null pointer access
> - inner classes that are public that should be default or protected (e.g. RegionHistoryInformation)
> - assignment to an input parameter

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message