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.

Sign up for GitHub

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

Jump to bottom

wildcards in index name #143

Closed
martijnvanbeers opened this issue Aug 14, 2016 · 4 comments
Closed

wildcards in index name #143

martijnvanbeers opened this issue Aug 14, 2016 · 4 comments
Labels
bug
Milestone
v0.7.4

Comments

@martijnvanbeers
Copy link

When I installed elasticdsl today, the elastic library seems to have gotten updated (to 0.7.2; unfortunately I have no idea what version I had before), and since then I get the following error when I try to use a * wildcard when specifying an index:

> tweets <- Search(index="tweet3-2016.*", asdf = T, size = 10000)$hits$hits$'_source' Error: 404 - IndexMissingException[[tweet3-2016.%2A] missing]

I know I can specify multiple indices, but that gets tedious fairly quickly if you have daily indices, so I'm hoping that either I missed a new way to do wildcards, or that this functionality can be brought back.
@sckott
Copy link
Contributor

sckott commented Aug 14, 2016

@martijnvanbeers thanks for your question. I'll have a look and get back to you

@sckott
Copy link
Contributor

sckott commented Aug 15, 2016

Hmmm, I originally used url escaping to make sure indexes and type's with special characters worked (#49 )

sckott added a commit that referenced this issue Aug 15, 2016
@sckott
remove esc() method call in Search and Search_uri as those calls duli…
0b9997e 
…cate

the url escape as its already done in internal fxn
added tests for wildcard usage as well as illegal characters, and special characters that are allowed
bumped dev version
#143
@sckott
Copy link
Contributor

sckott commented Aug 15, 2016

@martijnvanbeers Okay, reinstall devtools::install_github("ropensci/elastic") then try again and let me know if it works

@sckott sckott added this to the v0.7.4 milestone Aug 15, 2016
@martijnvanbeers
Copy link
Author

@sckott yes, that works. awesome, thanks!

@sckott sckott added the bug label Aug 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
v0.7.4
Development

No branches or pull requests
2 participants
@sckott @martijnvanbeers