Limit the number of features drawn

LocalData / localdata-tiles

A tileserver for LocalData

5 stars 2 forks source link

Limit the number of features drawn #111

Closed prashtx closed 10 years ago

prashtx commented 10 years ago

We shouldn't support an unbounded number of features, or we have no way of limiting our memory usage.

20,000 features in one tile typically means we're drawing pretty dense features, so we shouldn't miss the extra features. But we can adjust this as needed through an environment variable to reflect:

the realities of usage
the memory limits of the hosting environment
memory-saving improvements we might make down the road

hampelm commented 10 years ago

Great! Just wanted to confirm that this PR should contain the streaming code. If so, good to go!

prashtx commented 10 years ago

Yup, the streaming stuff is intentional. It ought to be a little better for concurrency. We're realistically giving up a little performance there, but the overall change makes a big impact in the cases that kill us right now. And the streaming + transform stuff is more idiomatic node.