Remove useless LIMIT_OPTION_DEFAULT value from LimitOption
authorAlvaro Herrera <[email protected]>
Sat, 16 Dec 2023 17:20:03 +0000 (18:20 +0100)
committerAlvaro Herrera <[email protected]>
Sat, 16 Dec 2023 17:20:03 +0000 (18:20 +0100)
During the development that led to commit 357889eb17bb, for a time we
had the value LIMIT_OPTION_DEFAULT, which was mostly but not completely
removed later on, before commit.  Complete the removal now.

Author: Zhang Mingli <[email protected]>
Discussion: https://postgr.es/m/59d61a1a-3858-475a-964f-24468c97cc67@Spark

src/backend/parser/gram.y
src/include/nodes/nodes.h

index f16bbd3cddeaa24154950a49894e8e197e33d85e..63f172e1757f148d9adc1731b89445733bfae35c 100644 (file)
@@ -18461,7 +18461,7 @@ insertSelectOptions(SelectStmt *stmt,
                     parser_errposition(exprLocation(limitClause->limitCount))));
        stmt->limitCount = limitClause->limitCount;
    }
-   if (limitClause && limitClause->limitOption != LIMIT_OPTION_DEFAULT)
+   if (limitClause)
    {
        if (stmt->limitOption)
            ereport(ERROR,
index 4c32682e4ce661671f7ff6f48eba2080f2bf27e6..a178d08727ab9aa282ef4db89cbc00146313479a 100644 (file)
@@ -440,7 +440,6 @@ typedef enum LimitOption
 {
    LIMIT_OPTION_COUNT,         /* FETCH FIRST... ONLY */
    LIMIT_OPTION_WITH_TIES,     /* FETCH FIRST... WITH TIES */
-   LIMIT_OPTION_DEFAULT,       /* No limit present */
 } LimitOption;
 
 #endif                         /* NODES_H */