Nicola:
Following along a long drawn out merge … Kevin reports a failure based on one of your commits:
https://github.com/geoserver/geoserver/commit/f8d205f688ab9b98bfe7d315f5a883190cdce67d
Can you talk about what you were accomplishing with this commit? Is it something we should revert on master and reapply after Kevin’s branch is merged?
–
Jody Garnett
With a bit more looking, I’m getting the same unit test failure when I build straight from master so it doesn’t actually seem to be related to my pull request.
Failed tests: testHarvestImageMosaicWithDirectory(org.geoserver.catalog.rest.CoverageStoreFileUploadTest): expected:<…-11-01T00:00:00.000Z[,2008-11-02T00:00:00.000Z]> but was:<…-11-01T00:00:00.000Z>
On 10 June 2014 17:35, Jody Garnett <jody.garnett@anonymised.com> wrote:
Nicola:
Following along a long drawn out merge … Kevin reports a failure based on one of your commits:
https://github.com/geoserver/geoserver/commit/f8d205f688ab9b98bfe7d315f5a883190cdce67d
Can you talk about what you were accomplishing with this commit? Is it something we should revert on master and reapply after Kevin’s branch is merged?
–
Jody Garnett
HPCC Systems Open Source Big Data Platform from LexisNexis Risk Solutions
Find What Matters Most in Your Big Data with HPCC Systems
Open Source. Fast. Scalable. Simple. Ideal for Dirty Data.
Leverages Graph Analysis for Fast Processing & Easy Data Exploration
http://p.sf.net/sfu/hpccsystems
Geoserver-devel mailing list
Geoserver-devel@anonymised.comsts.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
–
Kevin Smith
Junior Software Engineer | Boundless
+1-778-785-7459
Hi all,
the test failure that Kevin reported should be caused by an older version of the gt-imagemosaic module, so I think Kevin should update the master branch of geotools in order to get the updates for that module. If the test failure continues to exist, I will investigate on it.
Anyway I noticed that the GeoServer Jenkins build is not broken now.
Best Regards,
Nicola.
2014-06-11 2:40 GMT+02:00 Kevin Smith <ksmith@anonymised.com>:
With a bit more looking, I’m getting the same unit test failure when I build straight from master so it doesn’t actually seem to be related to my pull request.
Failed tests: testHarvestImageMosaicWithDirectory(org.geoserver.catalog.rest.CoverageStoreFileUploadTest): expected:<…-11-01T00:00:00.000Z[,2008-11-02T00:00:00.000Z]> but was:<…-11-01T00:00:00.000Z>
–
Ing. Nicola Lagomarsini
Junior Software Engineer
GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054 Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
http://www.geo-solutions.it
http://twitter.com/geosolutions_it
On 10 June 2014 17:35, Jody Garnett <jody.garnett@anonymised.com> wrote:
Nicola:
Following along a long drawn out merge … Kevin reports a failure based on one of your commits:
https://github.com/geoserver/geoserver/commit/f8d205f688ab9b98bfe7d315f5a883190cdce67d
Can you talk about what you were accomplishing with this commit? Is it something we should revert on master and reapply after Kevin’s branch is merged?
–
Jody Garnett
HPCC Systems Open Source Big Data Platform from LexisNexis Risk Solutions
Find What Matters Most in Your Big Data with HPCC Systems
Open Source. Fast. Scalable. Simple. Ideal for Dirty Data.
Leverages Graph Analysis for Fast Processing & Easy Data Exploration
http://p.sf.net/sfu/hpccsystems
Geoserver-devel mailing list
Geoserver-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geoserver-devel
–
Kevin Smith
Junior Software Engineer | Boundless
+1-778-785-7459
On Wed, Jun 11, 2014 at 9:57 AM, Nicola Lagomarsini <
nicola.lagomarsini@anonymised.com> wrote:
Hi all,
the test failure that Kevin reported should be caused by an older version
of the gt-imagemosaic module, so I think Kevin should update the master
branch of geotools in order to get the updates for that module. If the test
failure continues to exist, I will investigate on it.Anyway I noticed that the GeoServer Jenkins build is not broken now.
Also verified locally, a full build of geotools and geoserver works fine
here
Cheers
Andrea
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
-------------------------------------------------------
OK, thanks, I’ll try clearing my maven repo and rebuilding.
On 11 June 2014 06:49, Andrea Aime <andrea.aime@anonymised.com> wrote:
–
Kevin Smith
Junior Software Engineer | Boundless
+1-778-785-7459
On Wed, Jun 11, 2014 at 9:57 AM, Nicola Lagomarsini <nicola.lagomarsini@anonymised.com> wrote:
Hi all,
the test failure that Kevin reported should be caused by an older version of the gt-imagemosaic module, so I think Kevin should update the master branch of geotools in order to get the updates for that module. If the test failure continues to exist, I will investigate on it.
Anyway I noticed that the GeoServer Jenkins build is not broken now.
Also verified locally, a full build of geotools and geoserver works fine here
Cheers
Andrea
–
==
GeoServer Professional Services from the experts! Visit
http://goo.gl/NWWaa2 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
OK, tests passing. Thanks.
On 11 June 2014 09:17, Kevin Smith <ksmith@anonymised.com> wrote:
OK, thanks, I’ll try clearing my maven repo and rebuilding.
–
Kevin Smith
Junior Software Engineer | Boundless
+1-778-785-7459
On 11 June 2014 06:49, Andrea Aime <andrea.aime@anonymised.com> wrote:
–
Kevin Smith
Junior Software Engineer | Boundless
+1-778-785-7459
On Wed, Jun 11, 2014 at 9:57 AM, Nicola Lagomarsini <nicola.lagomarsini@anonymised.com> wrote:
Hi all,
the test failure that Kevin reported should be caused by an older version of the gt-imagemosaic module, so I think Kevin should update the master branch of geotools in order to get the updates for that module. If the test failure continues to exist, I will investigate on it.
Anyway I noticed that the GeoServer Jenkins build is not broken now.
Also verified locally, a full build of geotools and geoserver works fine here
Cheers
Andrea
–
==
GeoServer Professional Services from the experts! Visit
http://goo.gl/NWWaa2 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