Skip to content

fix(comms): add "recover" function in ecl/workunit#4619

Merged
GordonSmith merged 1 commit into
hpcc-systems:mainfrom
jeclrsg:workunit-add-recover
Jul 1, 2026
Merged

fix(comms): add "recover" function in ecl/workunit#4619
GordonSmith merged 1 commit into
hpcc-systems:mainfrom
jeclrsg:workunit-add-recover

Conversation

@jeclrsg

@jeclrsg jeclrsg commented Jun 30, 2026

Copy link
Copy Markdown
Contributor
  • expose a "recover" convenience function in ecl/workunit
  • fix the flags passed to WUResubmit by the "resubmit" function

Those flags being passed to WUResubmit appear to have always been wrong? When comparing the oldest history of ecl/workunit.ts (

return this.WUResubmit({
CloneWorkunit: false,
ResetWorkflow: false
}).then(() => {
) with the oldest history of src/ESPWorkunit.ts in the HPCC-Platform (https://github.com/hpcc-systems/HPCC-Platform/blob/40d2bf091dbfdc588ce7d5378bddce4d787b525b/esp/src/src/ESPWorkunit.ts#L384)

Checklist:

  • The commit message is properly formatted and free of typos.
    • The commit message title makes sense in a changelog, by itself.
    • The commit message includes a "fixes" reference if appropriate.
    • The commit is signed.
  • The change has been fully tested:
    • I have viewed all related gallery items
    • I have viewed all related dermatology items
  • There are no similar instances of the same problem that should be addressed
    • I have addressed them here
    • I have raised new issues to address them separately

Testing:

* expose a "recover" convenience function in ecl/workunit
* fix the flags passed to WUResubmit by the "resubmit" function

Signed-off-by: Jeremy Clements <79224539+jeclrsg@users.noreply.github.com>
@jeclrsg jeclrsg requested a review from GordonSmith June 30, 2026 15:54
@GordonSmith GordonSmith merged commit d78ca60 into hpcc-systems:main Jul 1, 2026
3 of 5 checks passed
@github-actions github-actions Bot mentioned this pull request Jul 1, 2026
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