[Gluster-devel] Glupy connection closing
Justin Clift
jclift at redhat.com
Tue Apr 30 02:57:29 UTC 2013
On 30/04/2013, at 3:45 AM, Justin Clift wrote:
> On 29/04/2013, at 11:00 PM, Jeff Darcy wrote:
>> On 04/29/2013 12:23 PM, Justin Clift wrote:
>>> Tried establishing a tcp connection during the translator _init_, then
>>> re-using it for every fop invocation. The very first time invovation
>>> works, which likely executes the init code too. Everything after that
>>> though gives "broken pipe" errors:
>>>
>>> ERROR: Error received at msg.sendall() time: [Errno 32] Broken pipe
>>>
>>> Code here, if it's helpful:
>>>
>>> https://github.com/justinclift/glupy/blob/glusterflow_sharing/glusterflowclient.py
>>>
>>> On IRC, Kaleb mentioned it just might be how this works. i.e. Python
>>> closing all connections at the end of each invocation
>>
>> That doesn't quite make sense to me. AIUI, the C part of the code
>> should still have a borrowed reference to the Translator object from
>> PyObject_CallObject, so it shouldn't be deleted. If not, that's easy to
>> fix. However, even if that were the case, __init__ would be called on
>> the new object and things would still work (albeit inefficiently). Have
>> you verified that __init__ is in fact being called multiple times? If
>> not, then it's a different kind of problem that we'll need to figure out.
Actually, let me clarify things as I understand them, just in case it helps.
The translator __init__ in my Glupy module seems to be called just once, when
the vol file is loaded.
Basing that on the statements I used to have in it like:
print 'In translator __init__() function'
That would only turn up in the output once (when loading vol file), whereas
the fop functions (ie lookup_fop()) are definitely called once per operation.
(this is why it seemed like a good idea to initialise the tcp connection
once in init, and then reuse it in the fop() functions)
Does that help?
+ Justin
--
Open Source and Standards @ Red Hat
twitter.com/realjustinclift
More information about the Gluster-devel
mailing list