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

Ted Yu commented on HBASE-6965:
-------------------------------

{code}
+ * It will decide to use the sun api or its own implementation
{code}
I think it's better to replace sun with Oracle.
{code}
+public class OSMXBean
{code}
Please add annotation for audience and stability for the above class.
{code}
+   * Check if the OS is unix. If using the IBM java runtime, this
+   * will only work for linux.
{code}
Do you need to mention IBM in the above javadoc ?
{code}
+  public boolean getUnix() {
{code}
Rename method to isUnix().
{code}
+  private Long getOSUnixMXBeanMethod (String mBeanMethodName)
{code}
Rename above method runUnixMXBeanMethod().

                
> Generic MXBean Utility class to support all JDK vendors
> -------------------------------------------------------
>
>                 Key: HBASE-6965
>                 URL: https://issues.apache.org/jira/browse/HBASE-6965
>             Project: HBase
>          Issue Type: Improvement
>          Components: build
>    Affects Versions: 0.94.1
>            Reporter: Kumar Ravi
>            Assignee: Kumar Ravi
>              Labels: patch
>             Fix For: 0.94.3
>
>         Attachments: HBASE-6965.patch
>
>
> This issue is related to JIRA 
> https://issues.apache.org/jira/browse/HBASE-6945. This issue is opened to 
> propose the use of a newly created generic 
> org.apache.hadoop.hbase.util.OSMXBean class that can be used by other 
> classes. JIRA HBASE-6945 contains a patch for the class 
> org.apache.hadoop.hbase.ResourceChecker that uses OSMXBean. With the 
> inclusion of this new class, HBase can be built and become functional with 
> JDKs and JREs other than what is provided by Oracle.
>  This class uses reflection to determine the JVM vendor (Sun, IBM) and the 
> platform (Linux or Windows), and contains other methods that return the OS 
> properties - 1. Number of Open File descriptors;  2. Maximum number of File 
> Descriptors.
>  This class compiles without any problems with IBM JDK 7, OpenJDK 6 as well 
> as Oracle JDK 6. Junit tests (runDevTests category) completed without any 
> failures or errors when tested on all the three JDKs.The builds and tests 
> were attempted on branch hbase-0.94 Revision 1396305.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to