From: "Amit Kapila" <amit.kapila16@gmail.com>
> On Wed, Jan 15, 2014 at 8:12 PM, MauMau <maumau307@gmail.com> wrote:
>> Thanks for reviewing and testing the patch. Yes, at first I did what you
>> mentioned, but modified the patch according to some advice in the mail
>> thread. During redo, create_tablespace_directories() needs to handle the
>> case where the $PGDATA/pg_tblspc/xxx is not a symlink but a directory
>> even
>> on UNIX/Linux. Please see TablespaceCreateDbspace is().
>> destroy_tablespace_directories() doesn't have to handle such situation.
>
> If create_tablespace_directories() needs to handle with directory both on
> Windows/Linux, then shouldn't it be a runtime check as in your first
> version rather than compile time check?
> Also isn't that the reason why destroy_tablespace_directories() have
> similar
> check?
I see..., and you are correct. The first version of my patch should be the
right fix. It seems that my head went somewhere when I submitted the second
revision.
What should I do? Should I re-submit the first revision as the latest fifth
revision and link the email from the CommitFest newest entry?
Regards
MauMau
