Items for Friday

Rod Widdowson rdw at steadingsoftware.com
Mon Jan 30 13:49:35 EST 2017


>> . We decided 60s is too long a timeout for the dynamic case, since
>> it's directly user-visible, being "inline" in the user request flow.
> 

I can buy that

> And the batch case has to handle large files that can take 30+ seconds in unusual cases to download, the dynamic one doesn't.

That too.

> Probably if one or the other is relying on the HttpClient default

That's what made my mind explode.  There was also the unfortunate setting of defaults in parsers which is unavoidable in this case, but something we avoid for our own beans.

> it might be better to have both metadata resolver parsers setting it explicitly, just so there's parity in handling it at that later, but that's style.

Rather more than that in that iirc we decided that the http client doesn't do what it is documented to do and if we want document what ours do we have to force the issue.

I would prefer seeing these values set explicitly by the parsers rather than defaulted, and i'd like it to be easy to find out what the default is.  Less important, but important to my mind is to put common function into common code just not to excess.

I'm not however claiming insight on how to make it so. Sorry..


More information about the dev mailing list