Hi all, I’d like to request that the JDBC Plugin for SAP HANA (modules/unsupported/jdbc-ng/jdbc-hana) gets promoted to a supported module.
With regards to the Gold Star Quality Assurance Check (Plugin QA Test) I’ve taken the following steps: * I’ve added module status files (NOTICE.txt, README.md, REVIEW.md) with https://github.com/geotools/geotools/pull/2213, but review is still missing, of course. * There are no known bugs. * I’ve uploaded some screenshots of the plugin being used in GeoServer to https://sap-my.sharepoint.com/:f:/p/stefan_uhrig/EpIpvp4ghn9Pthe020gkP5wBQXyte0U1kNpFt2hXt1t-6Q?e=tprHsR. I created some layers on OpenStreetMap data of the county I’m living in. I’m not much of a cartographer, but it should hopefully suffice to show that the plugin can work with larger amounts of data. * Documentation is available from http://docs.geotools.org/latest/userguide/library/jdbc/hana.html. I will act as module maintainer and answer questions regarding the module. There is also a Jacoco coverage report at https://sap-my.sharepoint.com/:f:/p/stefan_uhrig/EpIpvp4ghn9Pthe020gkP5wBQXyte0U1kNpFt2hXt1t-6Q?e=tprHsR (I tried to attach it to the mail, but the mailing list rejected attachments). Unit-test line coverage is about 89%, branch coverage about 81%. I’d appreciate if someone could do the IP check. Are there any further steps that are necessary to become a supported module? If you have any questions, don’t hesitate to ask. Thanks and best regards, Stefan From: Andrea Aime <andrea.a...@geo-solutions.it<mailto:andrea.a...@geo-solutions.it>> Sent: Freitag, 21. September 2018 15:34 To: Uhrig, Stefan <stefan.uh...@sap.com<mailto:stefan.uh...@sap.com>> Cc: Geotools-Devel list <geotools-devel@lists.sourceforge.net<mailto:geotools-devel@lists.sourceforge.net>> Subject: Re: [Geotools-devel] JDBC Plugin for SAP HANA On Fri, Sep 21, 2018 at 2:46 PM Uhrig, Stefan <stefan.uh...@sap.com<mailto:stefan.uh...@sap.com>> wrote: I don't have time to check the PR right now, but let me answer the quick and easy part :-) We would like our module to become an officially supported one. I would act as module maintainer. Can you help us with the necessary steps? The steps are documented in these two documents: http://docs.geotools.org/latest/developer/procedures/supported.html http://docs.geotools.org/latest/developer/procedures/check.html The instructions are a bit outdated, code coverage cannot be verified anymore with cobertura since the switch to Java 8, but you can use Jacoco or you IDE of choice to deliver a report. 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 di Montramito 3/A 55054 Massarosa (LU) phone: +39 0584 962313 fax: +39 0584 1660272 mob: +39 339 8844549 http://www.geo-solutions.it http://twitter.com/geosolutions_it ------------------------------------------------------- Con riferimento alla normativa sul trattamento dei dati personali (Reg. UE 2016/679 - Regolamento generale sulla protezione dei dati “GDPR”), si precisa che ogni circostanza inerente alla presente email (il suo contenuto, gli eventuali allegati, etc.) è un dato la cui conoscenza è riservata al/i solo/i destinatario/i indicati dallo scrivente. Se il messaggio Le è giunto per errore, è tenuta/o a cancellarlo, ogni altra operazione è illecita. Le sarei comunque grato se potesse darmene notizia. This email is intended only for the person or entity to which it is addressed and may contain information that is privileged, confidential or otherwise protected from disclosure. We remind that - as provided by European Regulation 2016/679 “GDPR” - copying, dissemination or use of this e-mail or the information herein by anyone other than the intended recipient is prohibited. If you have received this email by mistake, please notify us immediately by telephone or e-mail.
_______________________________________________ GeoTools-Devel mailing list GeoTools-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel