-
Notifications
You must be signed in to change notification settings - Fork 206
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
Harvester is failing to parse harvest object id strings correctly #405
Comments
Our CKAN instance relies heavily on harvesting, so running into harvest jobs that have to be manually aborted is troubling. After upgrading to version 1.3.0, I am seeing fetch_err.log with ERROR messages like the following:
Then after I manually abort the job, because giving a
So I would guess that somewhere in the code, a string is being passed when a list is expected. Or something. Am I really the only person who has run into this issue? |
I have the exact same issue, tested on We have a custom harvester, on every run it creates 2
Notice the
So |
I am going to keep adding comments, I hope that someone finds them useful. I can certainly try to answer any questions that developers may have. The error pattern seems somewhat random and interleaved with successful harvesting of certain records. Here is an example from today:
At this point the harvesting job is stuck, and no further harvesting jobs appear to start because giving the "harvester run" command does not put this job in a finished state. When this happens, I have begun performing the ritual of going through the following manual steps:
I don't know if all of these steps are necessary. If someone knows of a better way to handle the harvest errors, please let me know. |
**EDIT: The error described in this comment has been fixed, but has led to different errors in the harvesting code (version 1.3.0) described in my next comment. **
I recently upgraded ckanext-harvest to use redis 3, but I am now seeing this:
This might be a related discussion: redis/redis-py#1071
Or it is possible that the older version of setuptools is to blame; not sure.
I'm happy to provide additional information if needed. I'm running CKAN 2.8.3, with the ckanext-harvest plugin updated to the commit on master from April 21, 2020.
The text was updated successfully, but these errors were encountered: