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

Do not process already processing feed #62

Open
arjanfrans opened this issue Nov 5, 2021 · 2 comments
Open

Do not process already processing feed #62

arjanfrans opened this issue Nov 5, 2021 · 2 comments

Comments

@arjanfrans
Copy link

I am currently having a problem that my monitoring tool is getting a lot of errors; I have a cron job running; that processes the feeds every minute. From what I understand what is happening here; if the feed is already processing, and is then being processed again, then it will throw the The feed is not in a valid state. State is processing exception.

In my opinion this shouldn't really be an exception. Throwing this exception can be avoided by not processing 'already processing' feeds, right?

image

image

@igormukhingmailcom
Copy link
Member

I believe this can be fixed this way:

  • (at plugin repository) Create findReadyToBeProcessed rather than findEnabled which return enabled records in proper state.
  • (at server side) Run processing not so frequently

@arjanfrans
Copy link
Author

I am aware that running it so frequently doesn't make sense; but I did that since I was still in a testing phase. Anyway, this should be fixed, since it is an exception that shouldn't really happen.

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