The following review has been posted through the commitfest application: make installcheck-world: tested, failed Implements feature: tested, failed Spec compliant: tested, failed Documentation: tested, failed
Content. The proposed function waited long to be implemented and it will be very useful. Personally I used slow workaround, when I needed the reverse of the array. Run. This patch applies cleanly to HEAD. All regression tests pass successfully against the patch. Format. The code formatted according to The Code Guidelines Documentation. The documentation is updated, the description of the function is added. From my point of view, it would be better to mention, that function returns updated array (does not updates it in place, as a reader can understand), but other array returning functions in the documentation has the same style (silently assume: reverse == return reversed array). Conclusion. +1 for commiter review
