Re: splitting plpython into smaller parts - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: splitting plpython into smaller parts
Date
Msg-id 1324238037.28578.4.camel@vanquo.pezone.net
Whole thread Raw
In response to Re: splitting plpython into smaller parts  (Jan Urbański <wulczer@wulczer.org>)
Responses Re: splitting plpython into smaller parts  (Jan Urbański <wulczer@wulczer.org>)
List pgsql-hackers
On tis, 2011-12-06 at 00:58 +0100, Jan Urbański wrote:
> Rebased against master after the SPI cursor patch has been committed.
> 
> The first patch removes SPI boilerplate from the cursor functions as
> well and the second patch creates a plpython_cursor.c file.
> 
> A side effect of creating a separate file for cursors is that I had to
> make PLy_spi_transaction_{begin,commit,abort} helper functions external
> since they're used both by regular SPI execution functions and the
> cursor functions.
> 
> They live the plpython_spi.c which is not an ideal place for them, but
> IMHO it's not bad either.

Committed now.

I moved a few more things around.  I split up the plpython.h header file
to create a separate header file for each .c file, so that the
hierarchical releationship of the modules is clearer.  (The only cases
of circular relationships should be caused by use of global variables.)
Also, I named the files that contain classes or modules more like they
are in the CPython source code, e.g., plpy_cursorobject.c.



pgsql-hackers by date:

Previous
From: Jesper Krogh
Date:
Subject: Re: Page Checksums
Next
From: Dimitri Fontaine
Date:
Subject: Re: Command Triggers