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

Add search indexes #1165

Merged
merged 2 commits into from
Oct 6, 2015
Merged

Add search indexes #1165

merged 2 commits into from
Oct 6, 2015

Conversation

tseaver
Copy link
Contributor

@tseaver tseaver commented Oct 1, 2015

Uses #1164 as a base.

@tseaver tseaver added the search label Oct 1, 2015
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 1, 2015
@dhermes
Copy link
Contributor

dhermes commented Oct 1, 2015

Ping me here when #1164 is in (or at least once that review is done)

@tseaver
Copy link
Contributor Author

tseaver commented Oct 6, 2015

@dhermes Rebased after merge of #1164.

@@ -17,6 +17,7 @@

from gcloud.client import JSONClient
from gcloud.search.connection import Connection
from gcloud.search.index import Index

This comment was marked as spam.

@dhermes
Copy link
Contributor

dhermes commented Oct 6, 2015

Sorry for the delayed review. I accidentally knocked this PR out of my task queue.

@dhermes
Copy link
Contributor

dhermes commented Oct 6, 2015

LGTM. (Would still like to know about CRUD ops on indexes)

tseaver added a commit that referenced this pull request Oct 6, 2015
@tseaver tseaver merged commit eb30c53 into googleapis:master Oct 6, 2015
@tseaver tseaver deleted the search-index branch October 6, 2015 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants