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

Batching should complain when cohort has been changed vs last extraction [Magalie] #1395

Closed
tznind opened this issue Aug 31, 2022 · 0 comments · Fixed by #1414
Closed

Batching should complain when cohort has been changed vs last extraction [Magalie] #1395

tznind opened this issue Aug 31, 2022 · 0 comments · Fixed by #1414

Comments

@tznind
Copy link
Contributor

tznind commented Aug 31, 2022

Is your feature request related to a problem? Please describe.
When performing a large batched extraction. I get half way through then realise I have the wrong cohort. I change the cohort and run the extraction the rest of the way.

This currently results in:

  • Half of your file having one cohort
  • Second half of your file having the new cohort

Describe the solution you'd like
When Extraction is a batch resume it should check the extraction logs to determine if the cohort has changed and complain if it has. ProposedFix for CheckResult may be 'to reset progress to null (resetting batching)'.

@tznind tznind changed the title Batching should complain when cohort has been changed vs last extraction Batching should complain when cohort has been changed vs last extraction [Magalie] Aug 31, 2022
@tznind tznind moved this from Todo to In Progress in RDMP ~October 3rd Release Sep 12, 2022
@tznind tznind mentioned this issue Sep 12, 2022
@tznind tznind moved this from In Progress to Done in RDMP ~October 3rd Release Sep 12, 2022
@tznind tznind moved this from Done to Demoed in RDMP ~October 3rd Release Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

1 participant