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

Refactor Jobs APIs in V3 #2145

Open
awharn opened this issue May 16, 2024 · 1 comment
Open

Refactor Jobs APIs in V3 #2145

awharn opened this issue May 16, 2024 · 1 comment
Labels
enhancement New feature or request priority-low Legit issue but cosmetic or nice-to-have v4 Prospective changes for v4

Comments

@awharn
Copy link
Member

awharn commented May 16, 2024

Some Jobs APIs have a lot of required and optional parameters that might be better suited to be provided in an interface.

Originally raised by @zFernand0 in #2142 (comment)

However, others just log a trace and pass parameters directly to a common function (i.e. submitJclNotify to submitJclNotifyCommon, and submitJob to submitJobCommon)

We may want to investigate if we should change those functions in V3 to just use the same parameters as the common methods, or remove the common methods in favor of those methods.

Originally raised in #2142 (comment)

@awharn awharn added enhancement New feature or request new The issue wasn't triaged yet labels May 16, 2024
Copy link

Thank you for raising this enhancement request.
The community has 90 days to vote on it.
If the enhancement receives at least 5 upvotes, it is added to our development backlog.
If it receives fewer votes, the issue is closed.

@awharn awharn added the v3 prospective changes for v3 label May 16, 2024
@JTonda JTonda added priority-low Legit issue but cosmetic or nice-to-have v4 Prospective changes for v4 and removed new The issue wasn't triaged yet v3 prospective changes for v3 labels May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority-low Legit issue but cosmetic or nice-to-have v4 Prospective changes for v4
Projects
Status: Low Priority
Development

No branches or pull requests

2 participants