[ https://issues.apache.org/jira/browse/IGNITE-5791?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16096118#comment-16096118 ]
Pavel Tupitsyn commented on IGNITE-5791: ---------------------------------------- [~chief] my comments: * All {{TODO}}s must include JIRA links, see https://cwiki.apache.org/confluence/display/IGNITE/Coding+Guidelines#CodingGuidelines-TODOs * {{Matrix.java}} has commented-out {{map}} method, please remove. * Argument validation is missing in some places. Internal code (example: {{CacheUtils}}) should use {{assert}}, public APIs (example: {{SparseBlockDistributedMatrix.times()}}, mtx can be null) must throw exceptions for invalid arguments. * {{checkRowIndex}}: Should we include rowSize in exception text instead of writing this to console? > Block matrix Introduction > ------------------------- > > Key: IGNITE-5791 > URL: https://issues.apache.org/jira/browse/IGNITE-5791 > Project: Ignite > Issue Type: Sub-task > Components: ml > Reporter: Yury Babak > Assignee: Yury Babak > Fix For: 2.2 > > > Introduction of sparce distributed block matrix with more efficient > multiplication rather than row/col distributed matrix. -- This message was sent by Atlassian JIRA (v6.4.14#64029)