Hi Pavel, Patch does look good to me. And found no issues as such.
However here are my optional suggestions: 1. Frankly, I did not like name of the function "row_to_json_pretty_choosy". Something like "row_to_json_pretty_ignore_nulls" seems better to me. 2. To use ignore nulls feature, I have to always pass pretty flag. Which seems weired. Since we do support named argument, can we avoid that? No idea how much difficult it is. If we have a default arguments to this function then we do not need one and two argument variations for this function as well. And we can use named argument for omitting the required one. Just a thought. Rest looks good to me. Thanks -- Jeevan B Chalke Principal Software Engineer, Product Development EnterpriseDB Corporation The Enterprise PostgreSQL Company
