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

pagination doesn't respect sleep time #28

Closed
jeremybmerrill opened this issue Dec 18, 2013 · 7 comments
Closed

pagination doesn't respect sleep time #28

jeremybmerrill opened this issue Dec 18, 2013 · 7 comments

Comments

@jeremybmerrill
Copy link
Contributor

for some reason.

It should.

@esagara
Copy link
Contributor

esagara commented Dec 19, 2013

Ah so I didn't realize I was posting in the wrong area. We are coming across this issue as well. From what I have been reading in the code, it looks like the sleep time will only work if stashing is disabled. That is the only time I can find resp_and_cache[:from_resource] being set to true as required here. At least that is my take from what I am reading in the get method. Would it make sense to write the sleep into the download_from_resource! method?

@jeremybmerrill
Copy link
Contributor Author

Lol no worries, it's whatever.

@jeremybmerrill
Copy link
Contributor Author

Ah figured it out

fix tk shorlty

@jeremybmerrill
Copy link
Contributor Author

bccd019#diff-9347901060f5e801766116fbb544a20dL80

The boolean here was flipped; literally telling it not to sleep when it should have told it to sleep. Silly Jeremy.

@esagara
Copy link
Contributor

esagara commented Dec 19, 2013

Hey, I am running an array of urls through the scraper with no pagination. Sleep time is being ignored. I am working on subclassing some stuff to work around this, but I thought you should know.

@jeremybmerrill
Copy link
Contributor Author

Hmm. Can you make a gist or something so I can test it?

Also, can you make sure you're pulling the latest master from github? I didn't push an updated gem yet.

@esagara
Copy link
Contributor

esagara commented Dec 19, 2013

I pulled the master after you updated. Will send you a gist shortly.

On Thu, Dec 19, 2013 at 11:38 AM, Jeremy B. Merrill <
notifications@github.com> wrote:

Hmm. Can you make a gist or something so I can test it?

Also, can you make sure you're pulling the latest master from github? I
didn't push an updated gem yet.


Reply to this email directly or view it on GitHubhttps://github.com//issues/28#issuecomment-30943796
.

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

No branches or pull requests

2 participants