I was going to raise that concern earlier, I'm not a massive fan of the direct mapping from class/define parameter to config setting. Normally I have a defined resource for each setting - and have a class take a hash and use create_resources. This means you get class-time settings and later on users can add settings outside of the class if they desire.

Whatever the implementation is, like you say @nicklewis this means you can add features to puppetdb without having to have the module always keep up. :+1: