Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Having #14

Merged
merged 2 commits into from
Sep 15, 2014
Merged

Having #14

merged 2 commits into from
Sep 15, 2014

Conversation

seanv507
Copy link
Contributor

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

@fjy
Copy link
Member

fjy commented Sep 15, 2014

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
Copy link
Contributor Author

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
#14 (comment).

@fjy
Copy link
Member

fjy commented Sep 15, 2014

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
#14 (comment).


Reply to this email directly or view it on GitHub
#14 (comment).

fjy added a commit that referenced this pull request Sep 15, 2014
@fjy fjy merged commit f959c70 into druid-io:master Sep 15, 2014
@xvrl xvrl modified the milestone: 0.2.2 Jul 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants