This repository was archived by the owner on Jun 17, 2020. It is now read-only.
Pull latest from source repo #2
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
aguereca
pushed a commit
that referenced
this pull request
Dec 13, 2019
* Adding circleci support * Trying to prevent sigkills according to https://discuss.circleci.com/t/builds-getting-killed-with-vague-message-received-signal-killed/10214/12 * Remove formatting, manage memory * Max memory take 2 * Trying to build everything in one machine and one shot * Adding echoes and fixing dir names * See if parallelized builds work fine and fix geowebcache main pom location * Trying to tighten memory a bit more while preserving parallel builds * Back to sequential build * Attempt to split into smaller commands and run down to GeoServer * Some issues with directories, seem they are intepreted as relative, attempt to fix #1 * Some issues with directories, seem they are intepreted as relative, attempt to fix #2 * Some issues with directories, seem they are intepreted as relative, attempt to fix #3 * Don't update snapshots, trying to use less memory in GS build * Don't update snapshots, trying to use less memory in GT build * More memory work and starting to add repository caching * Fixing indentation * More caching fixes * Ok, trying to get offline costs too much in terms of downloads * One more attempt at getting a GS build that does not fail * Temporarily add early cache saving to have something for the next build (speed up trial and error roundtrip) * Damn you yaml * Switching all builds to batch mode * Cache was saved once, good enough, removing * Community modules have been compiled, without tests, in the first command, the third is redundant * Trying harder to release memory to the OS * Breaking install into two lines, it keeps on failing due to lack of memory... * Trying to centralize JVM configurations * Yet another attempt to control memory usage * war assembly failed in GWC due to memory, trying to give a bit more * Fixing typo * One more push on memory * Circle CI limits by the minute, try to reduce build time as much as possible
aguereca
pushed a commit
that referenced
this pull request
Dec 13, 2019
…s inside SLD filters
… across all instances
…crs:EPSG::4326) (#3166) * [GEOT-6714] Invalid URN parsing of version less URN SRS (urn:ogc:def:crs:EPSG::4326) - fixed URN parser - empty version is treated as NULL * [GEOT-6714] Andrea requested changes - using StringUtils method to identify empty version
Bumps [junit](https://github.com/junit-team/junit4) from 4.11 to 4.13.1. - [Release notes](https://github.com/junit-team/junit4/releases) - [Changelog](https://github.com/junit-team/junit4/blob/main/doc/ReleaseNotes4.11.md) - [Commits](junit-team/junit4@r4.11...r4.13.1) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [junit](https://github.com/junit-team/junit4) from 4.11 to 4.13.1. - [Release notes](https://github.com/junit-team/junit4/releases) - [Changelog](https://github.com/junit-team/junit4/blob/main/doc/ReleaseNotes4.11.md) - [Commits](junit-team/junit4@r4.11...r4.13.1) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* centralise JUnit versioning and bump to 4.13.1 * replace deprecated test methods * fix formatting in css module
Not quite sure how many of the inilliJ warnings needed to be addressed. Maven build now compiles without warning.
Update online testing doc with a note about running a Db2 docker container
[GEOT-6818] update jaiext to 1.1.20
…3338) * [GEOT-6647] incorporate jackson jts code and update to latest jts
* [GEOT-6799] WMTS online tests * Avoid size() > 0.
…ates for image/png outputformat
…ut of MongoDB bounds
[GEOT-6822][GEOT-6823][GEOT-6824] Update JDBC drivers: PostgreSQL (to 42.2.19), MS SQL Server (to 9.2.0.jre8), Oracle (to 19.10.0.0)
[GEOT-6826] Upgrade mockito version to 3.8.0 and unify declarations
…s calls * [GEOT-6808] - Http headers set for WMTS isn't used for GetCapabilities calls * Explaining the changes in upgrade documents * Some changes to the text * Make the PR compile Co-authored-by: Andrea Aime <andrea.aime@gmail.com>
Fix typo and remove double semicolon
* [GEOT-6650] Add limit on number of decimal places in coordinates
* [GEOT-6827] Upgrade to JTS Topology Suite 1.18.1 Signed-off-by: Jim Hughes <jhughes@ccri.com> * Fixing PMD error. Signed-off-by: Jim Hughes <jhughes@ccri.com> * Update modules/plugin/jdbc/jdbc-sqlserver/src/main/java/org/geotools/data/sqlserver/reader/SqlServerBinaryReader.java Co-authored-by: Mark Prins <1165786+mprins@users.noreply.github.com> * Update modules/plugin/jdbc/jdbc-sqlserver/src/main/java/org/geotools/data/sqlserver/reader/SqlServerBinaryReader.java * no return after throw Co-authored-by: Jim Hughes <jhughes@ccri.com> Co-authored-by: Ian Turton <ijturton@gmail.com> Co-authored-by: Mark Prins <1165786+mprins@users.noreply.github.com>
…t-ant-1.10.9 Bump ant from 1.9.15 to 1.10.9
…lue present but no M value GEOT-6740 fixed a buffer underflow exception when reading shapefile data - the problem was the code tried to read the M-value data regardless of whether M data was present in the shapefile. However, the fix was incomplete - it only checked there was one more double value remaining. The code would actually try to read a lot more M data than that. An example would be if the buffer contained a large shape (e.g. a circle) that was followed by a much smaller shape (e.g. a rectangle). In this case there's still more data remaining in the buffer, but not enough to match the numPoints in the shape. The fix is to check there as *at least* the expected amount of M-value data still remaining in the buffer before trying to read it. Test data files were updated so that they trigger the exception. Note that the order of shapes can be important - we usually want a large shape followed by a small shape. - zpolygons.shp: added a circle followed by a rectangle - zmultipoints.shp: added a large multipoint shape, followed by a single point. - zlines.shp: added a line with a lot of points, followed by a short line. Note the test data currently has both a polygonz.shp (single shape) and zpolygons.shp (multiple shapes), and likewise for lines and multipoints. This problem needed multiple shapes present in the shapefile, so I've amended those files. I've also added them back to the test case assertions, as the test code was no longer actually using them.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
20 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.