Skip to content
This repository has been archived by the owner on Sep 28, 2022. It is now read-only.

Require minimum args to bitmap operations through explicit check inst… #28

Merged

Conversation

alanbernstein
Copy link
Contributor

…ead of through function signature

@alanbernstein alanbernstein requested a review from yuce May 25, 2017 19:29
@coveralls
Copy link

coveralls commented May 25, 2017

Coverage Status

Coverage decreased (-0.3%) to 99.692% when pulling dc58eb8 on alanbernstein:bitmap-op-required-args into 0f97474 on pilosa:master.

@coveralls
Copy link

coveralls commented May 25, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 7b94a13 on alanbernstein:bitmap-op-required-args into 0f97474 on pilosa:master.

@coveralls
Copy link

coveralls commented May 25, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling af4cd1b on alanbernstein:bitmap-op-required-args into 0f97474 on pilosa:master.

@coveralls
Copy link

coveralls commented May 25, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 6221fe8 on alanbernstein:bitmap-op-required-args into 3f6ec51 on pilosa:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 6221fe8 on alanbernstein:bitmap-op-required-args into 3f6ec51 on pilosa:master.

@yuce
Copy link
Contributor

yuce commented May 25, 2017

This looks good!

@alanbernstein alanbernstein merged commit 2b73979 into FeatureBaseDB:master May 25, 2017
@alanbernstein alanbernstein deleted the bitmap-op-required-args branch May 25, 2017 21:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants