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

Pavel Tupitsyn commented on IGNITE-3170:
----------------------------------------

[~nathanchan2020] please see some comments on GitHub.

As for tests, I don't think we need tests for ToString overrides. However, a 
test to check that all public types have a ToString override can be useful. 
I've pushed a commit to your branch with such a test - please review and let me 
know what you think. 

> .NET: Add user-friendly ToString overrides for public types
> -----------------------------------------------------------
>
>                 Key: IGNITE-3170
>                 URL: https://issues.apache.org/jira/browse/IGNITE-3170
>             Project: Ignite
>          Issue Type: Improvement
>          Components: platforms
>    Affects Versions: 1.1.4
>            Reporter: Pavel Tupitsyn
>            Assignee: Kin (Nathan) Chan
>            Priority: Major
>              Labels: .net, newbie
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> Events, cache entry events, mutable cache entries, etc, should have 
> {{ToString}} overridden. See {{CacheEntry}} for example.
> {{IgniteConfiguration}} (and inner property classes) can be displayed as XML.



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

Reply via email to