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

fear: add onChildFailure option #2710

Merged
merged 6 commits into from
Aug 23, 2024
Merged

fear: add onChildFailure option #2710

merged 6 commits into from
Aug 23, 2024

Conversation

roggervalf
Copy link
Collaborator

No description provided.

@roggervalf roggervalf changed the base branch from v6 to master August 15, 2024 05:17
@roggervalf roggervalf changed the base branch from master to v6 August 15, 2024 05:18
@roggervalf roggervalf changed the base branch from v6 to master August 17, 2024 01:44
@@ -126,12 +114,13 @@ if rcall("EXISTS", jobIdKey) == 1 then -- // Make sure job exists
-- 2) move the job Id to parent "processed" set
-- 3) push the results into parent "results" list
-- 4) if parent's dependencies is empty, then move parent to "wait/paused". Note it may be a different queue!.
if parentId == "" and parentKey ~= "" then
parentId = getJobIdFromKey(parentKey)
parentQueueKey = getJobKeyPrefix(parentKey, ":" .. parentId)
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this logic is not needed anymore as all parent data is saved in parent attribute

local function moveParentIfNeeded(parentData, parentKey, jobIdKey,
failedReason, timestamp)
if parentData['ocf'] then
if parentData['ocf'] == 'fail' then
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can possible have a mapping like:

fail: f
ignore: i
remove: r
wait: w

that way we can only compare 1 character

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sounds good to me

@roggervalf roggervalf changed the base branch from master to v6 August 17, 2024 03:31
@roggervalf roggervalf changed the title fix: restore fpof with default value as true fear: add onChildFailure option Aug 17, 2024
@roggervalf roggervalf changed the base branch from v6 to master August 20, 2024 02:32
@roggervalf roggervalf changed the base branch from master to v6 August 20, 2024 05:41
@roggervalf roggervalf merged commit 8542941 into v6 Aug 23, 2024
10 checks passed
@roggervalf roggervalf deleted the fpof branch August 23, 2024 23:38
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

Successfully merging this pull request may close these issues.

2 participants