Re: [PATCH] Support negative indexes in split_part

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

Patch looks good to me. Seems like a useful feature, and I agree that the two-pass implementation makes the change very
easyto review.
 

Quick note on test coverage: gcov marks the "needle not found" branch (the one marked `/* special case if fldsep not
foundat all */`) as being completely uncovered. I don't think that needs to gate this patch; it looks like it was
uncoveredbefore this feature was added.
 

Doc builds are currently failing due to what appears to be an xmllint failure:

    /usr/bin/xmllint --path . --noout --valid postgres.sgml
    error : Unknown IO error
    postgres.sgml:21: warning: failed to load external entity "http://www.oasis-open.org/docbook/xml/4.5/docbookx.dtd"

but that doesn't have anything to do with this patch. Marking Ready for Committer. (I'm a little new to this myself, so
someoneplease let me know if I'm jumping the gun.)
 

Thanks!
--Jacob

The new status of this patch is: Ready for Committer