Re: [Geotools-devel] GeoTools 23-RC ready for testing - part II the return of testing

2020-03-10 Thread Jody Garnett 
Some notes from testing the project download:

1) README.md links
- broken link to
https://docs.geotools.org/stable/developer/procedures/contribute.html
- broken link to:
https://docs.geotools.org/stable/developer/procedures/pull_requests.html

2) VERSION.txt

Does not have the correct substitutions (or indeed any substitutions):

version: ${project.version}
git revision: ${build.commit.id}
build date: ${build.timestamp}


3) target/VERSION.txt is fine

4) Builds from PDT (UTC -7) completed
--
Jody Garnett


On Tue, 10 Mar 2020 at 20:27, Jody Garnett  wrote:

> Build is once again complete, please test artifacts and report back:
> -
> https://build.geoserver.org/view/release/job/geotools-release/230/artifact/build/release/distribution/23-RC/
>
> Andrea if testing is successful please proceed with geowebcache, or I will
> catch it tomorrow after work.
> --
> Jody Garnett
>
___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

[Geotools-devel] GeoTools 23-RC ready for testing - part II the return of testing

2020-03-10 Thread Jody Garnett 
Build is once again complete, please test artifacts and report back:
-
https://build.geoserver.org/view/release/job/geotools-release/230/artifact/build/release/distribution/23-RC/

Andrea if testing is successful please proceed with geowebcache, or I will
catch it tomorrow after work.
--
Jody Garnett
___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Re: [Geotools-devel] geopkg issue

2020-03-10 Thread Jody Garnett 
Thanks David, reviewing now - I have some questions we can discuss on the
ticket.
(My own understanding of geopackage is no doubt the limiting factor here)
--
Jody Garnett


On Tue, 10 Mar 2020 at 14:17, David Blasby  wrote:

> I put together a pull request - very minor change.  I also put in a test
> case that messes with the timezone -- hopefully this will make sure this
> doesn't happen again.
>
> https://github.com/geotools/geotools/pull/2837
>
> Thanks for the help/ideas/code Andrea! Couldn't have done it without you!
>
> dave
>
> On Tue, Mar 10, 2020 at 11:19 AM Andrea Aime 
> wrote:
>
>> I've found  the flags related to local date handling:
>>
>> https://github.com/geotools/geotools/blob/029dc5c8ccc503de3cfd7f4d1c939d8056c260c4/modules/library/metadata/src/main/java/org/geotools/util/factory/GeoTools.java#L239
>>
>> https://github.com/geotools/geotools/blob/029dc5c8ccc503de3cfd7f4d1c939d8056c260c4/modules/library/metadata/src/main/java/org/geotools/util/factory/Hints.java#L992
>>
>> However they seem to be affecting only GML encoding, not JDBC code
>> directly.
>> I'm guessing that in JDBC land java.sql.Date fields are treated as
>> without a time zone? Not sure.
>> See also this page of documentation about PostgreSQL:
>> https://www.postgresql.org/docs/9.1/datatype-datetime.html
>> Quoting: "The SQL standard requires that writing just timestamp be
>> equivalent to timestamp without time zone"
>>
>> Cheers
>> Andrea
>>
>>
>> On Tue, Mar 10, 2020 at 7:01 PM David Blasby 
>> wrote:
>>
>>> I'll look into the problem this afternoon.
>>>
>>> The timezone handling in sql lite is very limited and confusing.
>>>
>>> Cheers,
>>> David
>>> ___
>>> GeoTools-Devel mailing list
>>> GeoTools-Devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>>
>>
>>
>> --
>>
>> Regards, Andrea Aime == 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
>
___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Re: [Geotools-devel] geopkg issue

2020-03-10 Thread David Blasby 
I put together a pull request - very minor change.  I also put in a test
case that messes with the timezone -- hopefully this will make sure this
doesn't happen again.

https://github.com/geotools/geotools/pull/2837

Thanks for the help/ideas/code Andrea! Couldn't have done it without you!

dave

On Tue, Mar 10, 2020 at 11:19 AM Andrea Aime 
wrote:

> I've found  the flags related to local date handling:
>
> https://github.com/geotools/geotools/blob/029dc5c8ccc503de3cfd7f4d1c939d8056c260c4/modules/library/metadata/src/main/java/org/geotools/util/factory/GeoTools.java#L239
>
> https://github.com/geotools/geotools/blob/029dc5c8ccc503de3cfd7f4d1c939d8056c260c4/modules/library/metadata/src/main/java/org/geotools/util/factory/Hints.java#L992
>
> However they seem to be affecting only GML encoding, not JDBC code
> directly.
> I'm guessing that in JDBC land java.sql.Date fields are treated as without
> a time zone? Not sure.
> See also this page of documentation about PostgreSQL:
> https://www.postgresql.org/docs/9.1/datatype-datetime.html
> Quoting: "The SQL standard requires that writing just timestamp be
> equivalent to timestamp without time zone"
>
> Cheers
> Andrea
>
>
> On Tue, Mar 10, 2020 at 7:01 PM David Blasby 
> wrote:
>
>> I'll look into the problem this afternoon.
>>
>> The timezone handling in sql lite is very limited and confusing.
>>
>> Cheers,
>> David
>> ___
>> GeoTools-Devel mailing list
>> GeoTools-Devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>
>
>
> --
>
> Regards, Andrea Aime == 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

Re: [Geotools-devel] geopkg issue

2020-03-10 Thread Andrea Aime 
I've found  the flags related to local date handling:
https://github.com/geotools/geotools/blob/029dc5c8ccc503de3cfd7f4d1c939d8056c260c4/modules/library/metadata/src/main/java/org/geotools/util/factory/GeoTools.java#L239
https://github.com/geotools/geotools/blob/029dc5c8ccc503de3cfd7f4d1c939d8056c260c4/modules/library/metadata/src/main/java/org/geotools/util/factory/Hints.java#L992

However they seem to be affecting only GML encoding, not JDBC code directly.
I'm guessing that in JDBC land java.sql.Date fields are treated as without
a time zone? Not sure.
See also this page of documentation about PostgreSQL:
https://www.postgresql.org/docs/9.1/datatype-datetime.html
Quoting: "The SQL standard requires that writing just timestamp be
equivalent to timestamp without time zone"

Cheers
Andrea


On Tue, Mar 10, 2020 at 7:01 PM David Blasby 
wrote:

> I'll look into the problem this afternoon.
>
> The timezone handling in sql lite is very limited and confusing.
>
> Cheers,
> David
> ___
> GeoTools-Devel mailing list
> GeoTools-Devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>


-- 

Regards, Andrea Aime == 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

[Geotools-devel] geopkg issue

2020-03-10 Thread David Blasby 
I'll look into the problem this afternoon.

The timezone handling in sql lite is very limited and confusing.

Cheers,
David
___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Re: [Geotools-devel] GeoTools 23-RC ready for testing

2020-03-10 Thread Ian Turton 
JDBC stores I think. This certainly sounds familiar

Ian

On Tue, 10 Mar 2020 at 10:05, Andrea Aime 
wrote:

> Btw, I think Ian at one point worked on a setting on whether to prefer
> local time zone vs GMT... but I don't
> quite remember what it is supposed to affect... GML encoders? Converters?
> Data stores?
>
> Cheers
> Andrea
>
>
> On Tue, Mar 10, 2020 at 10:48 AM Andrea Aime 
> wrote:
>
>> Hi Jody,
>> tried with a local build with no remote repo, the GeoPackage tests fail,
>> seems a timezone issue
>>
>>
>> ---
>> Test set: org.geotools.geopkg.GeoPkgDatetimeTest
>>
>> ---
>> Tests run: 11, Failures: 3, Errors: 0, Skipped: 0, Time elapsed: 0.186 s
>> <<< FAILURE! - in org.geotools.geopkg.GeoPkgDatetimeTest
>> testMax(org.geotools.geopkg.GeoPkgDatetimeTest)  Time elapsed: 0.086 s
>>  <<< FAILURE!
>> java.lang.AssertionError: expected:<2020-02-23> but was:<2020-02-22>
>> at
>> org.geotools.geopkg.GeoPkgDatetimeTest.testMax(GeoPkgDatetimeTest.java:156)
>>
>> testMin(org.geotools.geopkg.GeoPkgDatetimeTest)  Time elapsed: 0.008 s
>>  <<< FAILURE!
>> java.lang.AssertionError: expected:<2020-02-19> but was:<2020-02-18>
>> at
>> org.geotools.geopkg.GeoPkgDatetimeTest.testMin(GeoPkgDatetimeTest.java:178)
>>
>> testGroupBy(org.geotools.geopkg.GeoPkgDatetimeTest)  Time elapsed: 0.008
>> s  <<< FAILURE!
>> java.lang.AssertionError: expected:<2020-02-19> but was:<2020-02-18>
>> at
>> org.geotools.geopkg.GeoPkgDatetimeTest.testGroupBy(GeoPkgDatetimeTest.java:206)
>>
>> I have a crazy day ahead of me, will try to have a look later but cannot
>> promise.
>>
>> Cheers
>> Andrea
>>
>> On Tue, Mar 10, 2020 at 8:32 AM Jody Garnett 
>> wrote:
>>
>>> Build is complete, please test artifacts and report back:
>>> - https://build.geoserver.org/view/release/job/geotools-release/229/
>>>
>>> Andrea if testing is successful please proceed with geowebcache, or I
>>> will catch it after work tomorrow if you are busy.
>>> --
>>> Jody Garnett
>>> ___
>>> GeoTools-Devel mailing list
>>> GeoTools-Devel@lists.sourceforge.net
>>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>>
>>
>>
>> --
>>
>> Regards, Andrea Aime == 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.*
>>
>
>
> --
>
> Regards, Andrea Aime == 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” -

Re: [Geotools-devel] GeoTools 23-RC ready for testing

2020-03-10 Thread Nuno Oliveira 
Hi all,
thx for the release Jody, got a green build here:
[INFO] Reactor Summary for GeoTools 23-RC:
[INFO] 
[INFO] GeoTools ... SUCCESS
[  0.768 s]
[INFO] Build tools for Geotools 2 . SUCCESS
[  0.044 s]
[INFO] Maven plugins for Geotools 2 ... SUCCESS
[  0.039 s]
[INFO] JAR files collector  SUCCESS
[  4.102 s]
[INFO] JJTree and JavaCC compilers plugin . SUCCESS
[  2.777 s]
[INFO] Geotools modules ... SUCCESS
[  0.033 s]
[INFO] Geotools libraries . SUCCESS
[  0.077 s]
[INFO] Open GIS Interfaces  SUCCESS
[  6.019 s]
[INFO] OGC Web Service Models . SUCCESS
[  0.035 s]
[INFO] Xlink Model  SUCCESS
[  1.617 s]
[INFO] Open Web Services Model  SUCCESS
[  2.224 s]
[INFO] Metadata ... SUCCESS
[  8.217 s]
[INFO] Sample data module . SUCCESS
[  4.667 s]
[INFO] Referencing services ... SUCCESS
[  5.822 s]
[INFO] Geotools plugins ... SUCCESS
[  0.040 s]
[INFO] EPSG Authority Service using HSQL database . SUCCESS
[  5.334 s]
[INFO] Main module  SUCCESS [
20.417 s]
[INFO] XML Parsing Support  SUCCESS [
11.332 s]
[INFO] Geotools extensions  SUCCESS
[  0.038 s]
[INFO] XML Parsing  SUCCESS
[  0.233 s]
[INFO] XML Parsing  SUCCESS
[  3.279 s]
[INFO] Feature Based Graphs and Networks .. SUCCESS
[  2.987 s]
[INFO] XML Code Generation Support  SUCCESS
[  0.832 s]
[INFO] Grid Coverage module ... SUCCESS [
24.981 s]
[INFO] EPSG Authority Service using WKT file .. SUCCESS
[  2.591 s]
[INFO] OGC CQL to Filter parser ... SUCCESS
[  2.940 s]
[INFO] Property File DataStore  SUCCESS
[  1.600 s]
[INFO] GeoTIFF grid coverage exchange module .. SUCCESS
[  3.161 s]
[INFO] JDBC DataStore Support . SUCCESS
[  4.917 s]
[INFO] JDBC DataStore Plugins . SUCCESS
[  0.018 s]
[INFO] H2 DataStore ... SUCCESS [
31.232 s]
[INFO] Oracle DataStore ... SUCCESS
[  0.545 s]
[INFO] PostGIS DataStore .. SUCCESS
[  2.207 s]
[INFO] Shapefile module ... SUCCESS
[  9.016 s]
[INFO] Feature transforming feature source wrapper  SUCCESS
[  2.501 s]
[INFO] WorldImage datasource module ... SUCCESS
[  2.046 s]
[INFO] imagemosaic datasource module .. SUCCESS [
20.298 s]
[INFO] ArcGrid datasource module .. SUCCESS [
17.744 s]
[INFO] GML2 XML Support ... SUCCESS
[  2.443 s]
[INFO] GML3 XML Support ... SUCCESS [
12.344 s]
[INFO] Filter XML Support . SUCCESS
[  3.212 s]
[INFO] SLD XML Support  SUCCESS
[  4.894 s]
[INFO] Render . SUCCESS
[01:09 min]
[INFO] Filter Encoding Specification Model  SUCCESS
[  1.686 s]
[INFO] Web Feature Service Model .. SUCCESS
[  1.358 s]
[INFO] Web Processing Service Model ... SUCCESS
[  1.581 s]
[INFO] Web Coverage Service Model . SUCCESS
[  1.813 s]
[INFO] Catalog Services for the Web Model . SUCCESS
[  1.406 s]
[INFO] Web Map Tile Service model . SUCCESS
[  5.326 s]
[INFO] Dynamic symbolizer module based on JFreeChart and Eastwood
SUCCESS [  1.992 s]
[INFO] Coverage Multidimensional Module ... SUCCESS
[  0.112 s]
[INFO] API interfaces . SUCCESS
[  3.115 s]
[INFO] NetCDF gridcoverage module . SUCCESS [
45.352 s]
[INFO] GRIB gridcoverage module ... SUCCESS [
20.195 s]
[INFO] CSVDataStore ... SUCCESS
[  2.636 s]
[INFO] Extensions to EPSG authority factory ... SUCCESS
[  2.370 s]
[INFO] EPSG Authority Service using PostgreSQL database ... SUCCESS
[  0.253 s]
[INFO] Feature-Pregeneralized . SUCCESS
[  1.800 s]
[INFO] OWS XML Support  SUCCESS
[  2.344 s]
[INFO] Filter Encoding Specification XML Support .. SUCCESS
[  3.808 s]
[INFO] GeoPackage Module .. SUCCESS [
18.177 s]

Re: [Geotools-devel] GeoTools 23-RC ready for testing

2020-03-10 Thread Andrea Aime 
Btw, I think Ian at one point worked on a setting on whether to prefer
local time zone vs GMT... but I don't
quite remember what it is supposed to affect... GML encoders? Converters?
Data stores?

Cheers
Andrea


On Tue, Mar 10, 2020 at 10:48 AM Andrea Aime 
wrote:

> Hi Jody,
> tried with a local build with no remote repo, the GeoPackage tests fail,
> seems a timezone issue
>
>
> ---
> Test set: org.geotools.geopkg.GeoPkgDatetimeTest
>
> ---
> Tests run: 11, Failures: 3, Errors: 0, Skipped: 0, Time elapsed: 0.186 s
> <<< FAILURE! - in org.geotools.geopkg.GeoPkgDatetimeTest
> testMax(org.geotools.geopkg.GeoPkgDatetimeTest)  Time elapsed: 0.086 s
>  <<< FAILURE!
> java.lang.AssertionError: expected:<2020-02-23> but was:<2020-02-22>
> at
> org.geotools.geopkg.GeoPkgDatetimeTest.testMax(GeoPkgDatetimeTest.java:156)
>
> testMin(org.geotools.geopkg.GeoPkgDatetimeTest)  Time elapsed: 0.008 s
>  <<< FAILURE!
> java.lang.AssertionError: expected:<2020-02-19> but was:<2020-02-18>
> at
> org.geotools.geopkg.GeoPkgDatetimeTest.testMin(GeoPkgDatetimeTest.java:178)
>
> testGroupBy(org.geotools.geopkg.GeoPkgDatetimeTest)  Time elapsed: 0.008 s
>  <<< FAILURE!
> java.lang.AssertionError: expected:<2020-02-19> but was:<2020-02-18>
> at
> org.geotools.geopkg.GeoPkgDatetimeTest.testGroupBy(GeoPkgDatetimeTest.java:206)
>
> I have a crazy day ahead of me, will try to have a look later but cannot
> promise.
>
> Cheers
> Andrea
>
> On Tue, Mar 10, 2020 at 8:32 AM Jody Garnett 
> wrote:
>
>> Build is complete, please test artifacts and report back:
>> - https://build.geoserver.org/view/release/job/geotools-release/229/
>>
>> Andrea if testing is successful please proceed with geowebcache, or I
>> will catch it after work tomorrow if you are busy.
>> --
>> Jody Garnett
>> ___
>> GeoTools-Devel mailing list
>> GeoTools-Devel@lists.sourceforge.net
>> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>>
>
>
> --
>
> Regards, Andrea Aime == 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.*
>


-- 

Regards, Andrea Aime == 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.*

Re: [Geotools-devel] GeoTools 23-RC ready for testing

2020-03-10 Thread Andrea Aime 
Hi Jody,
tried with a local build with no remote repo, the GeoPackage tests fail,
seems a timezone issue

---
Test set: org.geotools.geopkg.GeoPkgDatetimeTest
---
Tests run: 11, Failures: 3, Errors: 0, Skipped: 0, Time elapsed: 0.186 s
<<< FAILURE! - in org.geotools.geopkg.GeoPkgDatetimeTest
testMax(org.geotools.geopkg.GeoPkgDatetimeTest)  Time elapsed: 0.086 s  <<<
FAILURE!
java.lang.AssertionError: expected:<2020-02-23> but was:<2020-02-22>
at
org.geotools.geopkg.GeoPkgDatetimeTest.testMax(GeoPkgDatetimeTest.java:156)

testMin(org.geotools.geopkg.GeoPkgDatetimeTest)  Time elapsed: 0.008 s  <<<
FAILURE!
java.lang.AssertionError: expected:<2020-02-19> but was:<2020-02-18>
at
org.geotools.geopkg.GeoPkgDatetimeTest.testMin(GeoPkgDatetimeTest.java:178)

testGroupBy(org.geotools.geopkg.GeoPkgDatetimeTest)  Time elapsed: 0.008 s
 <<< FAILURE!
java.lang.AssertionError: expected:<2020-02-19> but was:<2020-02-18>
at
org.geotools.geopkg.GeoPkgDatetimeTest.testGroupBy(GeoPkgDatetimeTest.java:206)

I have a crazy day ahead of me, will try to have a look later but cannot
promise.

Cheers
Andrea

On Tue, Mar 10, 2020 at 8:32 AM Jody Garnett  wrote:

> Build is complete, please test artifacts and report back:
> - https://build.geoserver.org/view/release/job/geotools-release/229/
>
> Andrea if testing is successful please proceed with geowebcache, or I will
> catch it after work tomorrow if you are busy.
> --
> Jody Garnett
> ___
> GeoTools-Devel mailing list
> GeoTools-Devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>


-- 

Regards, Andrea Aime == 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

Re: [Geotools-devel] proposal: reduce proposal time limit to ten days

2020-03-10 Thread Ian Turton 
+0 on length

+1 on a limit

Ian

On Tue, 3 Mar 2020 at 19:36, Jody Garnett  wrote:

> Our current proposal time limit was set in the days of subversion when we
> had a single "trunk" and relatively strong divisions between modules in our
> library. The idea being that a 3-day waiting period was good enough for
> feedback before starting work :)
>
> To avoid stagnation by lack of interest/time from community members the
> following assurances are provided:
>
>
>- svn access for changes is granted within 3 days from the proposal
>- proposal is accepted ‘automatically’ within 15 days (unless
>objections are raised)
>
> Now that we have GitHub and pull requests developers can work more
> independently "svn access" is no longer a bottleneck, however it is still
> easy to let proposals alone for days without feedback (and then be
> surprised when the pull request comes in.
>
> I would like to propose reducing our "accepted automatically" time to 10
> days (giving two work weeks, and two weekends for PMC feedback).
>
> I think we can also start sharing proposals with the user list
>
> Here is what that change looks like:
>
> To avoid stagnation by lack of interest/time from community members:
>
>
>- please share work-in-progress as a draft pull request for early
>   feedback
>   - community feedback and welcomed and encouraged
>   - any remaining proposal PMC votes considered +0  ‘automatically’
>   after 10 days
>
>
> --
> Jody Garnett
> ___
> GeoTools-Devel mailing list
> GeoTools-Devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/geotools-devel
>


-- 
Ian Turton
___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

[Geotools-devel] GeoTools 23-RC ready for testing

2020-03-10 Thread Jody Garnett 
Build is complete, please test artifacts and report back:
- https://build.geoserver.org/view/release/job/geotools-release/229/

Andrea if testing is successful please proceed with geowebcache, or I will
catch it after work tomorrow if you are busy.
--
Jody Garnett
___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Re: [Geotools-devel] GeoTools 23.x stable created

2020-03-10 Thread Jody Garnett 
Small update, GeoTools 23-RC revision is
98252c14d12485497354e91b617646269534057c
(Had to commit the README and documentation links to "stable")
--
Jody Garnett


On Mon, 9 Mar 2020 at 23:38, Jody Garnett  wrote:

> Release candidate is underway, thanks for patience on this:
> - The 23.x stable branch 
> has been created, along with jira, and build.geosever.org jobs
> .
> - Master is updated to 24-SNAPSHOT
> - I added an ant build/release.xml covering some of the sed scripts,
> release instructions updated to reflect this
>
> Next up is the 23-RC (from the new 23.x branch) using the following
> revisions:
>
>- GeoTools *23-RC*
>   - 9c8b82bdfec32f6a8d0ac88bf699ae2762c1338a
>- GeoWebCache *1.17-RC*
>   - 6ea93e250e94ce93e6793a4f33c14dc2ca934194
>- GeoServer *2.17-RC*
>   - bb149a93694b9a8eafe7b964c9825ebe2d76a862
>
> --
> Jody Garnett
>
___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

[Geotools-devel] GeoTools 23.x stable created

2020-03-10 Thread Jody Garnett 
Release candidate is underway, thanks for patience on this:
- The 23.x stable branch 
has been created, along with jira, and build.geosever.org jobs
.
- Master is updated to 24-SNAPSHOT
- I added an ant build/release.xml covering some of the sed scripts,
release instructions updated to reflect this

Next up is the 23-RC (from the new 23.x branch) using the following
revisions:

   - GeoTools *23-RC*
  - 9c8b82bdfec32f6a8d0ac88bf699ae2762c1338a
   - GeoWebCache *1.17-RC*
  - 6ea93e250e94ce93e6793a4f33c14dc2ca934194
   - GeoServer *2.17-RC*
  - bb149a93694b9a8eafe7b964c9825ebe2d76a862

--
Jody Garnett
___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel