On Sun, Nov 16, 2025 at 11:18 PM Neil Chen <[email protected]> wrote: > After your patch changes, the line '*opid = InvalidOid;' seems removable. > > Also, if the second validation check of opclass after 'get_opclass_method' > feels a bit odd, moving 'get_opclass_opfamily_and_input_type' to the very top > would work -- purely for visual clarity. ：）
Thanks for the review! Here is a patch with your suggestions incorporated.
Yours,
--
Paul ~{:-)
[email protected]
v2-0001-Improve-comment-and-error-handling-in-GetOperator.patch
Description: Binary data
