Re: BUG #16663: DROP INDEX did not free up disk space: idle connection hold file marked as deleted

The following review has been posted through the commitfest application:
make installcheck-world:  tested, passed
Implements feature:       tested, passed
Spec compliant:           not tested
Documentation:            not tested

Hi, I have tested the feature and it worked well. 
One thing that doesn't matter is that the modify here seems unnecessary, right?

> mdunlinkfork(RelFileNodeBackend rnode, ForkNumber forkNum, bool isRedo)
> {
> char       *path;
> -    int            ret;
> +    int            ret = 0;
> path = relpath(rnode, forkNum