Dmitriy, I think it should stay under "Compute Grid" block because Deployment SPI have the meaning only for ComputeTasks.
Yakov, yes, Valentine has reviewed the jira and changed page name on: https://apacheignite.readme.io/docs/deployment-spi. I will fix your comments and will update the jira. Thanks for comments! -- Artem -- On Wed, Sep 30, 2015 at 10:54 AM, Yakov Zhdanov <yzhda...@apache.org> wrote: > I am looking at this page - > https://apacheignite.readme.io/docs/deployment-spi. Correct? > > Good job! However, I have minor comments: > > 1. SPI parameters explanations and code example should be before scanners > (scanner is SPI config as well, just more detailed info) > 2. code example should include all scanners + add spring configuration > example > 3. > >> quot > Http Uri Example > The following example will scan ignite/deployment folder on site > www.mysite.com using authentication 'username:password' every '10000' > milliseconds. > http://username:password;freq=10...@www.mysite.com:110/ignite/deployment > >>/quot > > I thought this scanner will download the page served by a server and > mysite.com:110 (that will be default page), parse it and download and > deploy all GAR files specified by href attributes of <a> elements on the > page. Please correct your content if you agree. > > Thanks! > > --Yakov > > 2015-09-30 4:41 GMT+03:00 Dmitriy Setrakyan <dsetrak...@apache.org>: > > > Artem, > > > > I would move the deployment SPI documentation as a bottom section in > "Zero > > Deployment" page. Also, please use this link for documentation: > > > > https://apacheignite.readme.io/docs > > > > D. > > > > On Tue, Sep 29, 2015 at 8:47 PM, Artem Shutak <ashu...@gridgain.com> > > wrote: > > > > > Folks, > > > > > > In process of working on [1] I've been created new article [2] on > > > readme.io > > > as long as this functionality has been missed in documentation. Please > > > review and feel free to comment. > > > > > > [1] https://issues.apache.org/jira/browse/IGNITE-1559 > > > [2] http://apacheignite.gridgain.org/docs/deployment-tasks > > > > > > -- Artem -- > > > > > >
