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

Fix to get parameters correctly when multiple job executions exist #4364

Closed
wants to merge 1 commit into from

Conversation

baezzys
Copy link
Contributor

@baezzys baezzys commented May 2, 2023

fixes #4246

@baezzys baezzys force-pushed the fix-job-parameters-bug branch 2 times, most recently from 13d8687 to f7a48c0 Compare May 2, 2023 11:20
@baezzys baezzys force-pushed the fix-job-parameters-bug branch from f7a48c0 to c89a82b Compare May 2, 2023 14:01
fmbenhassine added a commit that referenced this pull request May 16, 2023
Add integration test

Related to #4246
@fmbenhassine
Copy link
Contributor

Great! LGTM. Rebased and merged as e329b61. I also added an integration test for the case, see 7f304a1.

Thank you for your contribution!

fmbenhassine added a commit that referenced this pull request May 16, 2023
Add integration test

Related to #4246
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect retrieval of job parameters in SimpleJobExplorer#getJobExecutions
2 participants