[
https://issues.apache.org/jira/browse/YUNIKORN-1102?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17502732#comment-17502732
]
Wilfred Spiegelenburg commented on YUNIKORN-1102:
-------------------------------------------------
I think we need a follow up on this one to do a similar cleanup for
{{app.GetTask()}}
{code:java}
func (app *Application) GetTask(taskID string) (interfaces.ManagedTask,
error){code}
The returned task is nil or the error is nil. We do not provide any further
detail by adding the error. [~Tseng Hsi-Huang] the changes are a bit more
spread out in tests etc, but not more difficult than what you have done here.
Feel free to file a Jira and work on it.
> shim context getTask error is ignored
> -------------------------------------
>
> Key: YUNIKORN-1102
> URL: https://issues.apache.org/jira/browse/YUNIKORN-1102
> Project: Apache YuniKorn
> Issue Type: Improvement
> Components: shim - kubernetes
> Reporter: Wilfred Spiegelenburg
> Assignee: Tseng Hsi-Huang
> Priority: Major
> Labels: newbie, pull-request-available
>
> The {{context.getTask()}} call returns a {{Task}} struct and an error. In all
> cases either the task is nil or the error is nil. The error returned is also
> fixed and does not provide any detail not known without it (i.e. "application
> is not found" which only partially covers the cases)
> We need to bring the call in line with _GetApplication_ and the _GetNode_ in
> the cache and just return the task and handle the nil return.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
