Thread: Unwanted LOG during recovery of DROP TABLESPACE REDO
I found and fixed a bug that causes recovery (crash recovery , PITR) to throw unwanted LOG message if the tablespace symlink is not found during the processing of DROP TABLESPACE redo.
LOG: could not remove symbolic link "pg_tblspc/16384": No such file or directory
To Reproduce the issue:
1. Start the server.
2. Create a tablespace.
3. Perform Checkpoint.
4. Drop tablespace.
5. Stop server using immediate mode.
6. Start server : At this stage, recovery throw log message as mentioned above.
Reason is that DROP TABLESPACE has already removed symlink and again it is being tried to remove during recovery.
As it is very much possible that DROP TABLESPACE was successful and cleaned up the file before server crashed. So this should be considered as valid scenario and no need to throw
any LOG in such case. In case of processing of CREATE TABLESPACE redo, same is already handled.
I will add this to 2014-08 CF for review.
Thanks and Regards,
Kumar Rajeev Rastogi
Attachment
Rajeev rastogi <rajeev.rastogi@huawei.com> writes:
> I found and fixed a bug that causes recovery (crash recovery , PITR) to throw unwanted LOG message if the tablespace
symlinkis not found during the processing of DROP TABLESPACE redo.
> LOG: could not remove symbolic link "pg_tblspc/16384": No such file or directory
I don't think that's a bug: it's the designed behavior. Why should we
complicate the code to not print a log message in a situation where
it's unclear if the case is expected or not?
regards, tom lane
On 7/16/14 4:33 PM, Tom Lane wrote: > Rajeev rastogi <rajeev.rastogi@huawei.com> writes: >> I found and fixed a bug that causes recovery (crash recovery , PITR) to throw unwanted LOG message if the tablespace symlinkis not found during the processing of DROP TABLESPACE redo. >> LOG: could not remove symbolic link "pg_tblspc/16384": No such file or directory > > I don't think that's a bug: it's the designed behavior. Why should we > complicate the code to not print a log message in a situation where > it's unclear if the case is expected or not? I agree with Tom here; this doesn't seem like an improvement. .marko
On 08/15/2014 12:31 PM, Marko Tiikkaja wrote: > On 7/16/14 4:33 PM, Tom Lane wrote: >> Rajeev rastogi <rajeev.rastogi@huawei.com> writes: >>> I found and fixed a bug that causes recovery (crash recovery , PITR) to throw unwanted LOG message if the tablespacesymlink is not found during the processing of DROP TABLESPACE redo. >>> LOG: could not remove symbolic link "pg_tblspc/16384": No such file or directory >> >> I don't think that's a bug: it's the designed behavior. Why should we >> complicate the code to not print a log message in a situation where >> it's unclear if the case is expected or not? > > I agree with Tom here; this doesn't seem like an improvement. Well, for comparison, we also silently ignore non-existent files when replaying a "DROP TABLE". I could go either way myself, but this is clearly a very minor thing, and we have two -1's, so I'm marking this as "Rejected" in the commitfest. Thanks anyway! - Heikki