You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hbase.apache.org by "Jurriaan Mous (JIRA)" <ji...@apache.org> on 2016/05/07 12:53:12 UTC

[jira] [Created] (HBASE-15795) Cleanup all classes in package org.apache.hadoop.hbase.ipc for code style

Jurriaan Mous created HBASE-15795:
-------------------------------------

             Summary: Cleanup all classes in package org.apache.hadoop.hbase.ipc for code style
                 Key: HBASE-15795
                 URL: https://issues.apache.org/jira/browse/HBASE-15795
             Project: HBase
          Issue Type: Improvement
            Reporter: Jurriaan Mous
            Assignee: Jurriaan Mous
            Priority: Minor


While making adjustments for the async client I also need to fix code styling on the relevant classes. I want to do it all at once for ipc package so it does not mess up the diffs of future changes. I left out the classes I already fixed in patch in HBASE-15793

* Fixed import order
* Added curly brackets to all one line if statements
* Simplified generics with diamond operators where applicable
* Made private fields final when they were able
* use try with resource where it could be used
* renamed variables which were obscuring class fields
* Added documentation to empty @ doc fields
* Removed unnecessary setting to null 
* Removed unused getDelimitedMessageAsByteBuffer and readChunked methods from private IPCUtil class



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)