Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

params not included in next_batch #27

Open
GoogleCodeExporter opened this issue Mar 13, 2015 · 6 comments
Open

params not included in next_batch #27

GoogleCodeExporter opened this issue Mar 13, 2015 · 6 comments

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. res = s.select(keyword, fields='id,pubDate')
2. res = res.next_batch()

What is the expected output? What do you see instead?
1. solr => path=/select params={fl=id,pubDate,score ...
2. solr => path=/select params={fl=*,score ...

(2) missing field list param

What version of the product are you using? On what operating system?
solrpy-0.9.4, Windows

Please provide any additional information below.


Original issue reported on code.google.com by otor...@gmail.com on 6 Apr 2011 at 9:44

@dwoo
Copy link

dwoo commented Feb 4, 2016

I'm having this problem too.

next_batch() also loses the "score" parameter from the original search, so it ends up defaulting to "True". The monkey-patch kinda works for me, but I had to add

score=False

@edsu
Copy link
Collaborator

edsu commented Feb 5, 2016

Is this related at all to #36? If someone has time to send a pull request I can merge it in and release it.

@spanishgum
Copy link

spanishgum commented Mar 24, 2017

If it means anything, I had this same issue in both versions for python 2 and 3. I actually ran a performance test on my data to show the difference it made and verify the problem. The monkey patch worked perfectly.

solr_call_fix

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants