I guess the intention was to test incomplete SELECT statements, not

missing semicolons.

I also added a SELECT statement without a target list.

Manfred Koizar

diff --git a/src/test/regress/expected/errors.out b/src/test/regress/expected/errors.out
index 3941cc47ad4ac4541ab86caa6335d8c07c8bad05..698ccc85654b1c3611b922cdef7dfa80467dc74b 100644 (file)
--- a/src/test/regress/expected/errors.out
+++ b/src/test/regress/expected/errors.out
@@ -2,7 +2,12 @@
 -- ERRORS
 --
 -- bad in postquel, but ok in postsql
-select 1
+select 1;
+ ?column? 
+----------
+        1
+(1 row)
+
 --
 -- UNSUPPORTED STUFF
  
@@ -16,10 +21,14 @@ select 1
 -- RETRIEVE
  
 -- missing relation name 
-select
+select;
+ERROR:  parser: parse error at or near ";" at character 7
 -- no such relation 
 select * from nonesuch;
-ERROR:  parser: parse error at or near "select" at character 10
+ERROR:  Relation "nonesuch" does not exist
+-- missing target list
+select from pg_database;
+ERROR:  parser: parse error at or near "from" at character 8
 -- bad name in target list
 select nonesuch from pg_database;
 ERROR:  Attribute "nonesuch" not found

diff --git a/src/test/regress/sql/errors.sql b/src/test/regress/sql/errors.sql
index db041fa9cbfcc4127ec28ef21dd49fa363c6bfba..dee19bc136f4f42a0283c0fe7872709352fc690d 100644 (file)
--- a/src/test/regress/sql/errors.sql
+++ b/src/test/regress/sql/errors.sql
@@ -3,7 +3,7 @@
 --
 
 -- bad in postquel, but ok in postsql
-select 1
+select 1;
 
 
 --
@@ -20,11 +20,13 @@ select 1
 -- RETRIEVE
  
 -- missing relation name 
-select
+select;
 
 -- no such relation 
 select * from nonesuch;
 
+-- missing target list
+select from pg_database;
 -- bad name in target list
 select nonesuch from pg_database;
 -- bad attribute name on lhs of operator