On 30/04/18 22:00, Peter Geoghegan wrote:
> On Mon, Apr 30, 2018 at 10:18 AM, Heikki Linnakangas <hlinnaka@iki.fi> wrote:
>> Perhaps that would be OK, if it was properly commented. But it's not
>> actually hard to make BufFileSize() work on shared files, too, so I think we
>> should do that.
>
> I agree that this could use a comment, but I don't see much point in
> adding the extra FileSeek(). The fact that fd.c is unwilling to track
> filesize for non-temp files (where "non-temp" means a
> PathNameOpenTemporaryFile()-returned/exported file) is due to
> vfdP->fileSize being involved in temp_file_limit enforcement. Maybe a
> FD_TEMP_FILE_LIMIT assertion within FileGetSize() would help?
Seems simpler to just make it work. It's not like the extra lseek() is
going to make any performance difference. Functions that work only under
certain conditions are annoying.
Pushed, thanks.
- Heikki
