Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GEOS-2420] Add support to limit WPS resource consuption #874

Merged
merged 2 commits into from Jan 15, 2015

Conversation

aaime
Copy link
Member

@aaime aaime commented Jan 9, 2015

See https://github.com/geoserver/geoserver/wiki/GSIP%20123%20WPS%20input%20and%20execution%20limits

Contains it all, configuration, enforcement code, user interface, and documentation changes.

* The range of acceptable values for numeric values
* The max multiplicity of repeatable inputs (think of contour extraction, where the number of levels affects the execution time)

Starting with GeoServer 2.7.0, GeoServer allows the administrator to configure said limits, and fail requests that
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we need to say the version number - it just makes the documentation look stale over time. Each page here says GeoServer 2.7.0 at the top.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We normally add version numbers because users do not pay attention to the version of the documentation and wonder why the functionality is not working in their own already installed (old) GeoServer. There are several instances of these statements in the docs.

@jodygarnett
Copy link
Member

I am good to go on this one Andrea, thanks for the hard work.

@@ -23,8 +23,10 @@ Execution and resource management options
Execution settings:

* *Connection timeout*: the number of seconds the WPS will wait before giving up on a remote HTTP connection used to retrieve complex inputs
* *Maximum asynchronous executions run parallel*: the maximum number of asynchronous processes that will run in parallel at a given time. The others will be queued
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bmmpxf did you want to review these docs as they come in?

aaime added a commit that referenced this pull request Jan 15, 2015
[GEOS-2420] Add support to limit WPS resource consuption
@aaime aaime merged commit aa46f2f into geoserver:master Jan 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants