[jira] [Commented] (IGNITE-5349) Document C++ Compute Grid

["Denis Magda (JIRA)" <ji...@apache.org>]

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

Denis Magda commented on IGNITE-5349:
-------------------------------------

[~isapego], thanks! Everything looks good to me except for this section:
https://apacheignite-cpp.readme.io/v2.0/docs/distributed-closures#section-broadcast-methods

You made the "Job Declaration" section the last one and it's not 100% clear how the whole example works. I would suggest showing the example in the same way you did for the continuos queries filter (https://apacheignite-cpp.readme.io/v2.0/docs/continuous-queries#section-remote-filter) - define filter, register it, execute the app code.

So, let's do the following:
* Start with the `HelloWorldJob` from the "Job Declaration" code snippet.
* Move to the `BinaryType<HelloWorldJob>` explaining why do we need it, how the job will be serialized and deserialized. Just tell a bit about the implementation details.
* Paste the broadcast code snippet.
* Paste the async broadcast example.

Follow the same approach for this section: https://apacheignite-cpp.readme.io/v2.0/docs/distributed-closures#section-call-and-run-methods

> Document C++ Compute Grid
> -------------------------
>
>                 Key: IGNITE-5349
>                 URL: https://issues.apache.org/jira/browse/IGNITE-5349
>             Project: Ignite
>          Issue Type: Sub-task
>          Components: documentation, platforms
>    Affects Versions: 2.0
>            Reporter: Denis Magda
>            Assignee: Igor Sapego
>              Labels: cpp, documentation
>             Fix For: 2.1
>
>
> In 2.1 we're introducing compute grid support for C++. It has to be properly documented.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)