Closed KenjiTakahashi closed 10 years ago
Looks good to me! Just left some minor comments. I can't merge tho.
Hi @KenjiTakahashi, all Metamarkets open-source projects are covered by the same CLA as Druid.
If you are contributing on behalf of an organization, please have your company fill out the corporate CLA and email it to us: https://github.com/metamx/druid/raw/master/DruidCorporateCLA.pdf
If you are working on your own behalf and agree to the CLA, responding with a comment of "I agree to the terms of the Metamarkets Individual CLA as it exists in the current master branch" is sufficient: https://github.com/metamx/druid/raw/master/DruidIndividualCLA.pdf
@fjy I have just sent you the CLA. Sorry for the delay, had to reach our Sheriff on this :-).
Since we plan to use it, I went away and implemented first piece for #11 myself :-).