Committed, with some minor changes: > > 1) The too_many_rows check in exec_stmt_execsql included the errhint > conditionally, depending on force_error variable > > force_error = stmt->strict || stmt->mod_stmt; > use_errhint = !force_error; > > That is, the hint was not included when force_error=true, which does not > seem quite necessary. Based on off-list discussion with Pavel this was > unnecessary, so the hint is now included always. > > 2) The comment in plpgsql.h still mentioned "compile-time" checks only, > but the new checks are run-time checks. So tweaked accordingly. > > 3) A couple of minor formatting / code style changes. >
Many thanks Pavel > > regards > > -- > Tomas Vondra http://www.2ndQuadrant.com > PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services >