Re: [PATCHES] Fix for large file support

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Zdenek Kotala <Zdenek(dot)Kotala(at)Sun(dot)COM>, Andrew Dunstan <andrew(at)dunslane(dot)net>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: [PATCHES] Fix for large file support
Date: 2007-04-27 00:54:51
Message-ID: 200704270054.l3R0spe17764@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches


This has been saved for the 8.4 release:

http://momjian.postgresql.org/cgi-bin/pgpatches_hold

---------------------------------------------------------------------------

Tom Lane wrote:
> [ redirecting to -hackers for wider comment ]
>
> Zdenek Kotala <Zdenek(dot)Kotala(at)Sun(dot)COM> writes:
> > Tom Lane wrote:
> > LET_OS_MANAGE_FILESIZE is good way. I think one problem of this option I
> > fixed. It is size of offset. I went thru the code and did not see any
> > other problem there. However, how you mentioned it need more testing. I
> > going to take server with large disk array and I will test it.
>
> > I would like to add --enable-largefile switch to configure file to
> > enable access to wide group of users. What you think about it?
>
> Yeah, I was going to suggest the same thing --- but not with that switch
> name. We already use enable/disable-largefile to control whether 64-bit
> file access is built at all (this mostly affects pg_dump at the moment).
>
> I think the clearest way might be to flip the sense of the variable.
> I never found "LET_OS_MANAGE_FILESIZE" to be a good name anyway. I'd
> suggest "USE_SEGMENTED_FILES", which defaults to "on", and you can
> turn it off via --disable-segmented-files if configure confirms your
> OS has largefile support (thus you could not specify both this and
> --disable-largefile).
>
> regards, tom lane
>
> ---------------------------(end of broadcast)---------------------------
> TIP 5: don't forget to increase your free space map settings

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruce Momjian 2007-04-27 01:00:27 Re: Reviewers Guide to Deferred Transactions/TransactionGuarantee
Previous Message Bruce Momjian 2007-04-27 00:50:17 Re: Reviewers Guide to Deferred Transactions/Transaction Guarantee

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2007-04-27 01:00:27 Re: Reviewers Guide to Deferred Transactions/TransactionGuarantee
Previous Message Bruce Momjian 2007-04-27 00:50:17 Re: Reviewers Guide to Deferred Transactions/Transaction Guarantee