On 15.08.2011 18:54, Magnus Hagander wrote:
On Mon, Aug 15, 2011 at 16:53, Tom Lane<t...@sss.pgh.pa.us> wrote:
Magnus Hagander<mag...@hagander.net> writes:
I also ran into a similar problem with some WAL macro definitions that
are in xlog_internal.h. I've moved them to xlogdefs.h in the attached
xlog.diff file. Does that seem ok as well, or should I move them
I don't like the idea of exposing those to frontends, either. What do
you actually *need* out of that, and why?
PrevLogSeg - which also brings in XLogSegsPerFile.
XLogFileSize - which brings in XLogSegsPerFile and XLogSegSize
PrevLogSeg should be self explaining, as should xlogfilename.
XLogFileSize is required by XLByteAdvance() which is already in
xlogdefs.h - so xlogdefs.h actually has a hidden dependency on
xlog_internal.h here today.
I can certainly separate those out, but it seemed more clean to move
the whole block they were in.
Perhaps we should change the protocol so that it explicitly says which
file the streamed piece of WAL belongs to. That way a client could write
it to the correct file without knowing about all those macros.
Sent via pgsql-hackers mailing list (email@example.com)
To make changes to your subscription: