search

druid-io / pydruid

A Python connector for Druid
Other
508 stars 199 forks source link

Having #14

Closed seanv507 closed 10 years ago

seanv507 commented 10 years ago

Added having clause for groupby. Did not document in readme file

fjy commented 10 years ago

Hi @seanv507, thank for you for the contribution! We just have a brief CLA we were hoping you could sign. 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

seanv507 commented 10 years ago

Hi I work for liquidm (formerly madvertise) so we have a CLA with you already? I would contribute through company. Hope all fine Sean On 15 Sep 2014 20:12, "fjy" notifications@github.com wrote:

Hi @seanv507 https://github.com/seanv507, thank for you for the contribution! We just have a brief CLA we were hoping you could sign. 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

— Reply to this email directly or view it on GitHub https://github.com/metamx/pydruid/pull/14#issuecomment-55632758.

fjy commented 10 years ago

Yup, thanks.

On Mon, Sep 15, 2014 at 1:11 PM, seanv507 notifications@github.com wrote:

Hi I work for liquidm (formerly madvertise) so we have a CLA with you already? I would contribute through company. Hope all fine Sean On 15 Sep 2014 20:12, "fjy" notifications@github.com wrote:

Hi @seanv507 https://github.com/seanv507, thank for you for the contribution! We just have a brief CLA we were hoping you could sign. 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

— Reply to this email directly or view it on GitHub https://github.com/metamx/pydruid/pull/14#issuecomment-55632758.

— Reply to this email directly or view it on GitHub https://github.com/metamx/pydruid/pull/14#issuecomment-55650663.