ignite-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (IGNITE-3711) .NET: PlatformRawMemory is not covered with tests
Date Fri, 19 Aug 2016 13:44:22 GMT

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

ASF GitHub Bot commented on IGNITE-3711:

GitHub user ptupitsyn opened a pull request:


    IGNITE-3711 .NET: PlatformRawMemory is not covered with tests

    * Remove unused code
    * Add asserts

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ptupitsyn/ignite ignite-3711

Alternatively you can review and apply these changes as the patch at:


To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #971
commit ed2434f959bdc3e2a217eb7304c9b247d0230786
Author: Pavel Tupitsyn <ptupitsyn@apache.org>
Date:   2016-08-19T13:41:02Z

    IGNITE-3711 .NET: PlatformRawMemory is not covered with tests:
    * Remove unused code
    * Add asserts


> .NET: PlatformRawMemory is not covered with tests
> -------------------------------------------------
>                 Key: IGNITE-3711
>                 URL: https://issues.apache.org/jira/browse/IGNITE-3711
>             Project: Ignite
>          Issue Type: Bug
>          Components: platforms
>    Affects Versions: 1.8
>            Reporter: Pavel Tupitsyn
>            Assignee: Pavel Tupitsyn
>              Labels: .net
>             Fix For: 1.8
> Coverage suite shows that PlatformRawMemory is not called. This happens since the merge
of IGNITE-2943, which changes error propagation mechanism.
> Need to check whether old mechanism, which uses PlatformRawMemory, is still needed. Either
remove it, or cover with test.

This message was sent by Atlassian JIRA

View raw message