public inbox for [email protected]  
help / color / mirror / Atom feed
From: Federico Di Gregorio <[email protected]>
To: Daniele Varrazzo <[email protected]>
Cc: [email protected]
Subject: Re: Integration of Psycopg with XTA
Date: Wed, 3 Oct 2018 15:03:52 +0200
Message-ID: <[email protected]> (raw)
In-Reply-To: <CA+mi_8aTsj2d+kyrz+86ZOZ8Srno0T5Px6w+SWqfw5iiSPBkcA@mail.gmail.com>
References: <[email protected]>
	<[email protected]>
	<CA+mi_8a5Sm=Zum+8uRX+gDLrzzEU+F-x5xjommxL5e7jLBxHkg@mail.gmail.com>
	<[email protected]>
	<CA+mi_8bZ-hxA2DTshOh15b1GdbET8+EVO5eh9N2qyyfvDgXKCw@mail.gmail.com>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<[email protected]>
	<CA+mi_8YBz5GZRJfqj3Jr+gA4tpR7yd1aYXiPGQU9Sb7hiXJPfA@mail.gmail.com>
	<[email protected]>
	<CA+mi_8aTsj2d+kyrz+86ZOZ8Srno0T5Px6w+SWqfw5iiSPBkcA@mail.gmail.com>

On 10/03/2018 02:52 PM, Daniele Varrazzo wrote:
> On Wed, Oct 3, 2018 at 1:44 PM Federico Di Gregorio<[email protected]>  wrote:
>> On 10/03/2018 02:37 PM, Daniele Varrazzo wrote:
>>> On Wed, Oct 3, 2018 at 1:28 PM Federico Di Gregorio<[email protected]>   wrote:
>>>
>>>>> This is interesting because PyCapsule is for sure a "clean" way to pass
>>>>> the pointer from Psycopg to XTA.
>>> Fog, maybe it's worth adding the capsules for PGconn and PGresult to
>>> the objects interface and document them? Can do that for psycopg 2.8.
>> If it works I'd add it (and document it as an "internal" extension point
>> used by other C-level modules) for 2.8, yes.
>>
>> By PGresult you mean adding it on the cursor, or the connection?
> Yes, i'd say a _pgconn attribute or _get_pgconn() method on the
> connection, a _pgresult thing on the cursor.
> 
> By the way a read-only attribute as per your implementation seems
> appropriate to me. Christian, why do you think a function would be
> better?

I guess the capsule is meant to be consumed by the other API and using a 
method makes is explicit that a new object is instantiated every time. I 
really don't care about this, let's do whatever is more Pythonic.

federico

-- 
Federico Di Gregorio                         [email protected]
DNDG srl                                                  http://dndg.it
  I did appreciate the irony that I was whining about encoding issues on
   a mailing list that was unable to show those chars, too.
                                  -- Antti S. Lankila to mono-devel-list@




view thread (22+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: Integration of Psycopg with XTA
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox