Skip to content

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.

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