Skip to content

no need to jobVaild in openSession #4157

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.

Sign up for GitHub

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

Jump to bottom
Closed
fjq123123 opened this issue Mar 29, 2025 · 0 comments · Fixed by #4158
Closed

no need to jobVaild in openSession #4157

fjq123123 opened this issue Mar 29, 2025 · 0 comments · Fixed by #4158
Labels
kind/question Categorizes issue related to a new question

Comments

@fjq123123
Copy link
Contributor

Please describe your problem in detail

It feels unnecessary to perform JobValid validation during the openSession phase. At this stage, the callback functions in the SSN are all empty, making the validation meaningless.

Any other relevant information

No response
@fjq123123 fjq123123 added the kind/question Categorizes issue related to a new question label Mar 29, 2025
@fjq123123 fjq123123 mentioned this issue Mar 29, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/question Categorizes issue related to a new question
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

delete job vaild action in openSession fjq123123/volcano
1 participant
@fjq123123