[Python-Dev] [ 1673007 ] urllib2 requests history + HEAD (original) (raw)
K.Danilov aka koder k.danilov at ratmirlabs.com
Tue Mar 13 12:13:21 CET 2007
- Previous message: [Python-Dev] [ 1673007 ] urllib2 requests history + HEAD
- Next message: [Python-Dev] assert(obj) in Py_DECREF and Py_INCREF?
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
From: Facundo Batista <facundo at taniquetil.com.ar <mailto:facundo at taniquetil.com.ar>> This patch was posted by "koder_ua".
I think that Request must have a "request type" parameters, so people can send "HEAD" requests easily.
But it seems to me that keeping a request history in the module is bad, because it can easily grow up to thousands and explode (a.k.a. consume too much memory).
Fo example, I have a web service, running 7x24, and opening another web service, with around 10 requests per second. This means, keeping the history (around 50bytes each request), 1.2 GB of RAM in only a month!
So, I'll close this patch as "Rejected", for this reason, if anyone raises objections.
Regards,
. Facundo
This is probably a misunderstanding. Request's history don't store in the "module".They store in two places:
- In Request object (for current request, so they would be destroyеd with it);
- In HTTPConnection object (while request redirects). In HTTPConnection history stores only for current served Request. Even if You use the same HTTPConnection for many Requests, they (HTTPConnection) clear history every time when new Request starts.
from httplib HTTPConnection.putrequest patched
str = '%s %s %s' % (method, url, self._http_vsn_str) self._output(str) self.sended_hdrs = [str] <<< previous history die here
Full history for all processed request didn't not stored in any place_.
P.S. This message may be duplicated - first copy i sent from gmail.com and it didn't reach mail list for some unknown for me reasons.
KDanilov aka koder(aka koder_ua)
- Previous message: [Python-Dev] [ 1673007 ] urllib2 requests history + HEAD
- Next message: [Python-Dev] assert(obj) in Py_DECREF and Py_INCREF?
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]