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... 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 ------------------------------------------------------------------------------ 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
