[Geoserver-devel] Unsupported Extension Request for MonitorDAO

GeoServer GeoServer Developer
1

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

2

On Mon, Sep 28, 2015 at 2:59 PM, Eric Robertson <rwgdrummer@anonymised.com>
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.

-------------------------------------------------------

3

I will be the stable maintainer.

My student is invested in improving the UI portion. We had to wrap it up quickly in the interest of the GSoC time constraints.

Changing the package name is fine.

I will submit signed agreement as documented here (https://raw.githubusercontent.com/openlayers/cla/master/icla.txt).

Thanks.

···

On Mon, Sep 28, 2015 at 9:07 AM, Andrea Aime <andrea.aime@anonymised.com> wrote:

On Mon, Sep 28, 2015 at 2:59 PM, Eric Robertson <rwgdrummer@anonymised.com> 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.

4

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.

···

On 28 September 2015 at 06:07, Andrea Aime <andrea.aime@anonymised.com> wrote:

Geoserver-devel mailing list
Geoserver-devel@anonymised.comsts.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

–
Jody Garnett

On Mon, Sep 28, 2015 at 2:59 PM, Eric Robertson <rwgdrummer@anonymised.com> 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.

5

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 <jody.garnett@anonymised.com…> 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 <andrea.aime@anonymised.com> wrote:

Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

On Mon, Sep 28, 2015 at 2:59 PM, Eric Robertson <rwgdrummer@anonymised.com> 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.

6

That agreement is for open layers, this is the geoserver project.

Our github repo has the details (also shown when you make a pull request):

https://github.com/geoserver/geoserver/blob/master/CONTRIBUTING.md

···

On 28 September 2015 at 09:14, Eric Robertson <rwgdrummer@anonymised.com> wrote:

I will be the stable maintainer.

My student is invested in improving the UI portion. We had to wrap it up quickly in the interest of the GSoC time constraints.

Changing the package name is fine.

I will submit signed agreement as documented here (https://raw.githubusercontent.com/openlayers/cla/master/icla.txt).

Thanks.

Geoserver-devel mailing list
Geoserver-devel@anonymised.comsts.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

–
Jody Garnett

On Mon, Sep 28, 2015 at 9:07 AM, Andrea Aime <andrea.aime@anonymised.com> wrote:

On Mon, Sep 28, 2015 at 2:59 PM, Eric Robertson <rwgdrummer@anonymised.com…> 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.

7

I would slot the project in here: https://github.com/geoserver/geoserver/tree/master/src/community

Other community modules contain a core and web split:

Or under a single directory:

···

On 28 September 2015 at 10:02, Eric Robertson <rwgdrummer@anonymised.com> wrote:

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?

–
Jody Garnett

On Mon, Sep 28, 2015 at 12:15 PM, Jody Garnett <jody.garnett@anonymised.com> 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 <andrea.aime@anonymised.com> wrote:

Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

On Mon, Sep 28, 2015 at 2:59 PM, Eric Robertson <rwgdrummer@anonymised.com> 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.