Attending

Andrea Aime

Torben Barsballe

Jody Garnett

Kevin Smith

Gabriel Roldan

Actions from last meeting:

   -

   N/A

Agenda

   -

   Release Update
   -

   CITE Testing
   -

   Unchecked warnings cleanup/diamond
   -

   Gabe update in microserve experiment



Release Update

Release announcements:

   -

   http://blog.geoserver.org/2020/07/21/geoserver-2-17-2-released/
   -

   https://geotoolsnews.blogspot.com/2020/07/geotools-232-released.html


Release was uneventful, which was *great*.

Also updated geoserver website to remove foss4g 2020 and fundraising note.
CITE Testing

There is a big PR to review:

   -

   https://github.com/geoserver/geoserver/pull/4285
   -

   Some noise from “validation tests” included it cite data directories
   (can we remove these)

General feedback:

   -

   devops focus on server
   -

   weaker for local testing
   -

   May be hard on windows (due to use of docker on windows being hard to
   setup)


Notes:

   -

   This is large for github, you need to checkout locally to review
   -

   Q: Are their instructions for the developers guide?


Action:

   -

   Jody to test this on macOS (where docker should function)
   -

   Ask on the email list for windows test, and any instructions...


Planning:

   -

   distributed code sprint to actually pass the tests, during the fall?
   -

   Jody has some time and can set this up

Unchecked warnings cleanup/diamond

Group QA activity from last meet, idea/inspiration from Andrea :)

Now a public branch:

   -

   https://github.com/geotools/geotools/commits/unchecked
   -

   https://github.com/geotools/geotools/pull/3054
   -

   Coordination spreadsheet:
   
https://docs.google.com/spreadsheets/d/1voHK_V8ZdbN-yqqUkqrl0rJsLjqiOjmVwJkm1Fjd0z4/edit?usp=drive_web&ouid=100149442879963462152
   -

      Request access to edit
      -

      Use to coordinate activity - only one person work on a module at a
      time :)
      -

      Although for API changes they may affect more than one module
      -

   Aside:
   -

      Andrea tried with sed script, a few things like anonymous inner
      classes do not work


Approach:


   1.

   Clean up warnings module by module



   1.

   Use diamond this is a good way to get involved in GeoTools, making the
   codebase much easier to read :)


BEFORE:

   List<PropertyDescriptor> properties = new
ArrayList<PropertyDescriptor>();

FIX:

   List<PropertyDescriptor> properties = new ArrayList<>();

Gabriel update in microserve experiment

Working on camptocamp repository:
https://github.com/camptocamp/geoserver-microservices

   -

   Break out into docker per service
   -

   Going to try JMS with JDBCConfig / JDBCStore (for diskless)
_______________________________________________
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to