ndimiduk commented on pull request #3663:
URL: https://github.com/apache/hbase/pull/3663#issuecomment-915394733


   Sorry to nit-pick, but if the build is explicitly private API, can it be its 
own top-level class, rather than being an inner class of the public API? I 
think it's confusing to downstream developers when we inter-mingle public and 
private API resources in the same class.
   
   otherwise, +1


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@hbase.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to