Instead of failing requests that return > maxResultSize, return maxResultSize documents and warn

[alechenninger]

No description provided.

[paterczm]

I'm concerned this will lead to unpredictable results. I think it's safer to fail; that way we will see the problem quickly.

[alechenninger]

Seeing the problem quickly doesn't need to be coupled to failing the request or not. We can log and alert for example. It changes whether customers are alerting us or we're doing it without them knowing.

Of course the "correctness" of the response suffers, but in the spirit of graceful degradation I think that may be preferable to failing in a potentially user facing way. It pushes the decision to the client to make that call. We'd be providing more options to deal with failure.

[alechenninger]

@paterczm Even adding a limit on result size in bytes is technically a breaking change, just like lowering the result size in documents. Returning partial responses makes adding / changing these limits less likely to cause serious user facing problems.