You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ignite.apache.org by "Alexey Zinoviev (Jira)" <ji...@apache.org> on 2020/06/26 10:20:00 UTC

[jira] [Commented] (IGNITE-9740) [ML] Remove IgniteThread wrapper from ml unit test EvaluatorTest (follow up to IGNITE-9711)

    [ https://issues.apache.org/jira/browse/IGNITE-9740?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17146194#comment-17146194 ] 

Alexey Zinoviev commented on IGNITE-9740:
-----------------------------------------

This test was removed, no IgniteThread usage in ML tests anymore

> [ML] Remove IgniteThread wrapper from ml unit test EvaluatorTest (follow up to IGNITE-9711)
> -------------------------------------------------------------------------------------------
>
>                 Key: IGNITE-9740
>                 URL: https://issues.apache.org/jira/browse/IGNITE-9740
>             Project: Ignite
>          Issue Type: Bug
>          Components: ml
>            Reporter: Oleg Ignatenko
>            Assignee: Alexey Zinoviev
>            Priority: Critical
>             Fix For: 2.9
>
>
> [EvaluatorTest|https://github.com/apache/ignite/blob/master/modules/ml/src/test/java/org/apache/ignite/ml/selection/scoring/evaluator/EvaluatorTest.java] involves {{IgniteThread}} which is in fact not needed there and should be removed.
> {{IgniteThread}} usage is a remainder / copy-paste from older tests and examples that were using API requiring it. This API has been removed and there is no need for wrapping like that anymore. For the reference on how to perform suggested cleanup check changes made to ml examples per IGNITE-9711.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)