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

Fix ORC Bloom Filter #921 #923

Merged
merged 1 commit into from Jun 7, 2019

Conversation

2 participants
@dilipkasana
Copy link
Member

commented Jun 5, 2019

Fix for #921 , changing key from StreamId to Integer

@cla-bot

This comment has been minimized.

Copy link

commented Jun 5, 2019

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@prestosql.io. For more information, see https://github.com/prestosql/cla.

@dain

This comment has been minimized.

Copy link
Member

commented Jun 5, 2019

This is commit is virtually the same as the first commit in #914. If you can sign the CLA (https://github.com/prestosql/cla), I'll push this one and then rebase #914 on that.

@dain

dain approved these changes Jun 5, 2019

@dilipkasana

This comment has been minimized.

Copy link
Member Author

commented Jun 6, 2019

This is commit is virtually the same as the first commit in #914. If you can sign the CLA (https://github.com/prestosql/cla), I'll push this one and then rebase #914 on that.

@dain I have sent the CLA signed.Please push the commit.

@cla-bot cla-bot bot added the cla-signed label Jun 6, 2019

@prestosql prestosql deleted a comment from cla-bot bot Jun 7, 2019

@prestosql prestosql deleted a comment from martint Jun 7, 2019

@dain dain merged commit 0663067 into prestosql:master Jun 7, 2019

2 checks passed

Travis CI - Pull Request Build Passed
Details
verification/cla-signed
Details

@dain dain added this to the 314 milestone Jun 7, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.