So, in all the other services in this module, I've intentionally  not subscribed to config files.  I find that subscribing to config files in puppet is only really useful for non-production systems.  I don't like automated restarts of production services unless they are all very robust and handle failures very well.  I wouldn't want a bug in this puppet module to cause downtime for HBase users.

Also, FYI, you never need to both subscribe and require the same file.  Subscribing will auto-require the file.