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

Enis Soztutar commented on HBASE-15771:
---------------------------------------

We are explicitly exposing PB classes from this patch. Is this intentional, or 
just that the corresponding pojo's are not implemented yet? Also you seem to be 
favoring {{Client}} as the entry point versus {{Connection}} as in the Java 
client's case. Having a "client" object to keep the cluster connection is more 
intuitive, but it will be a deviation from our java API (not that it is a 
show-stopper).  Otherwise looks good. 

> Document all the public classes
> -------------------------------
>
>                 Key: HBASE-15771
>                 URL: https://issues.apache.org/jira/browse/HBASE-15771
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Elliott Clark
>            Assignee: Elliott Clark
>         Attachments: HBASE-15771.patch
>
>
> Add on doxygen comments for all public classes and methods.
> Once that's done then we can turn on doxygen with error on missing.



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

Reply via email to