[ 
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)

Reply via email to