Makes sense. For clarity, are you recommending a fork of geoserver, adding this extension under the 'extension' structure? Other than structural concerns, I can also assume application of the 'Eclipse' formatter for Java and XML.
Open to ideas for an extension folder name (not gsmonitor). The separation between 'web' and 'core' was used to clearly separate one as an SPI implementer of the monitor, the other a Wicket. They are not co-dependent. Although, I could easily keep them under one project folder, constructing different build artifacts. Thoughts? Using the extension name as ds for this discussion, the folder structure: extension/monitor/ds One glaring deficiency in the tool is that lack of internationalization. The H2 extension has a support for quite a few European based languages. Do you see this as something that needs addressing prior to integration? On Mon, Sep 28, 2015 at 12:15 PM, Jody Garnett <[email protected]> wrote: > I prefer code is refactored to match the project structure, it gives it > the option of becoming supported (i.e. meeting the coding style guidelines) > in the future. > > -- > Jody Garnett > > On 28 September 2015 at 06:07, Andrea Aime <[email protected]> > wrote: > >> On Mon, Sep 28, 2015 at 2:59 PM, Eric Robertson <[email protected]> >> wrote: >> >>> As part of the Google Summer of Code, through LocationTech, my student >>> (Joao) has extended the MonitorDAO to write Map Request Data as Simple >>> Features to a Data Store. The intent is to perform geo-spatial analytics >>> on map requests, such trending regions of interest. The design does not >>> assume the Data Store has been configured with GeoServer, as long as the >>> Data Store Factory is resolved through the SPI. A UI component (Wicket) >>> permits an administrator to configure the extension with a configured Data >>> Store. >>> >>> It would be nice to add to this project to an unsupported (or, >>> eventually, supported) extension. >>> >>> https://github.com/joaomartins27396/GSOC2015-gsmonitoext >>> >> >> Ah, interesting. To add community modules, you just need to ask, and we >> give you commit access (after the contribution agreement to OSGeo are >> signed). >> For supported land, well, that requires a bit more effort and a stable >> mantainer for the module... would that be you, your student, or you were >> hoping >> someone else to jump in? >> >> I also believe the packages should be switched to org.geoserver, but I'm >> not actually sure that's a strict requirement, we just don't >> have anything outside org.geoserver. >> >> Cheers >> Andrea >> >> >> -- >> == >> GeoServer Professional Services from the experts! Visit >> http://goo.gl/it488V for more information. >> == >> >> Ing. Andrea Aime >> @geowolf >> Technical Lead >> >> GeoSolutions S.A.S. >> Via Poggio alle Viti 1187 >> 55054 Massarosa (LU) >> Italy >> phone: +39 0584 962313 >> fax: +39 0584 1660272 >> mob: +39 339 8844549 >> >> http://www.geo-solutions.it >> http://twitter.com/geosolutions_it >> >> *AVVERTENZE AI SENSI DEL D.Lgs. 196/2003* >> >> Le informazioni contenute in questo messaggio di posta elettronica e/o >> nel/i file/s allegato/i sono da considerarsi strettamente riservate. Il >> loro utilizzo è consentito esclusivamente al destinatario del messaggio, >> per le finalità indicate nel messaggio stesso. Qualora riceviate questo >> messaggio senza esserne il destinatario, Vi preghiamo cortesemente di >> darcene notizia via e-mail e di procedere alla distruzione del messaggio >> stesso, cancellandolo dal Vostro sistema. Conservare il messaggio stesso, >> divulgarlo anche in parte, distribuirlo ad altri soggetti, copiarlo, od >> utilizzarlo per finalità diverse, costituisce comportamento contrario ai >> principi dettati dal D.Lgs. 196/2003. >> >> >> >> The information in this message and/or attachments, is intended solely >> for the attention and use of the named addressee(s) and may be confidential >> or proprietary in nature or covered by the provisions of privacy act >> (Legislative Decree June, 30 2003, no.196 - Italy's New Data Protection >> Code).Any use not in accord with its purpose, any disclosure, reproduction, >> copying, distribution, or either dissemination, either whole or partial, is >> strictly forbidden except previous formal approval of the named >> addressee(s). If you are not the intended recipient, please contact >> immediately the sender by telephone, fax or e-mail and delete the >> information in this message that has been received in error. The sender >> does not give any warranty or accept liability as the content, accuracy or >> completeness of sent messages and accepts no responsibility for changes >> made after they were sent or for other risks which arise as a result of >> e-mail transmission, viruses, etc. >> >> ------------------------------------------------------- >> >> >> ------------------------------------------------------------------------------ >> >> _______________________________________________ >> Geoserver-devel mailing list >> [email protected] >> https://lists.sourceforge.net/lists/listinfo/geoserver-devel >> >> >
------------------------------------------------------------------------------
_______________________________________________ Geoserver-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geoserver-devel
