On Thu, Jun 14, 2012 at 4:51 AM, Andrea Aime
<[email protected]>wrote:

> On Wed, Jun 6, 2012 at 7:51 AM, Justin Deoliveira <[email protected]>
> wrote:
> > Ok, found some spare cycles to read through this. And while I don't have
> a
> > lot of experience to draw from with WPS but I thin i like the second
> > interface as well, that is not separating special methods for changed
> > execution and try to handle them with a single interface.
> >
> > What type of object gets passed in as the executionId?
>
> It's a string, it is the identifier for all WPS requests and the one that
> gets
> returned when doing an asynch one. Internally it's built as a UUID.
>
> > One possibily would
> > be to roll a new "Execution" sort of descriptor class that could maintain
> > the actual id of the process, and other execution info such whether the
> > process is actually a changed one or not. Would also leave the door open
> to
> > more info specially about how the process is executed without having to
> > modify the methods further.
>
> I like the approach. I guess ExecutionEvent is probably a better name for
> a parameter of a listener?
> And it woudl be the only parameter of all methods, just that at a different
> point in the execution workflow it would have different kind of values
> attached.
> We could actually have a single callback method and specify the phase
> as a ExecutionEvent attribute... but then the listeners would have to do a
> if/switch inside the method...
>

Sounds good, ExecutionEvent makes sense. I do think it would be nicer to
have separate methods for different points of execution if possible, seems
to jive a little more with the rest of our "callback interfaces". But no
strong opinion.

>
> Opinions?
>
> Cheers
> Andrea
>
> Cheers
> Andrea
>
> --
> Ing. Andrea Aime
> GeoSolutions S.A.S.
> Tech lead
>
> Via Poggio alle Viti 1187
> 55054  Massarosa (LU)
> Italy
>
> phone: +39 0584 962313
> fax:      +39 0584 962313
> mob:    +39 339 8844549
>
> http://www.geo-solutions.it
> http://geo-solutions.blogspot.com/
> http://www.youtube.com/user/GeoSolutionsIT
> http://www.linkedin.com/in/andreaaime
> http://twitter.com/geowolf
>



-- 
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to