Also support LOGGED & UNLOGGED syntax for sequences

Refs #107

Author: nene

src/cst/Sequence.ts

@@ -44,7 +44,9 @@ export type SequenceOption =
   | SequenceOptionCycle
   | SequenceOptionNoCycle
   | SequenceOptionOwnedBy
-  | SequenceOptionSequenceName;
+  | SequenceOptionSequenceName
+  | SequenceOptionLogged
+  | SequenceOptionUnlogged;
 
 export interface SequenceOptionAsType extends BaseNode {
   type: "sequence_option_as_type";
@@ -123,6 +125,16 @@ export interface SequenceOptionSequenceName extends BaseNode {
   name: EntityName;
 }
 
+export interface SequenceOptionLogged extends BaseNode {
+  type: "sequence_option_logged";
+  loggedKw: Keyword<"LOGGED">;
+}
+
+export interface SequenceOptionUnlogged extends BaseNode {
+  type: "sequence_option_unlogged";
+  unloggedKw: Keyword<"UNLOGGED">;
+}
+
 export interface AlterSequenceStmt extends BaseNode {
   type: "alter_sequence_stmt";
   alterKw: Keyword<"ALTER">;

src/parser.pegjs

@@ -4200,6 +4200,8 @@ sequence_option
   / sequence_option_no_cycle
   / sequence_option_owned_by
   / sequence_option_sequence_name
+  / sequence_option_logged
+  / sequence_option_unlogged
 
 sequence_option_as_type
   = kw:(AS __) type:data_type {
@@ -4283,6 +4285,16 @@ sequence_option_sequence_name
     return loc({ type: "sequence_option_sequence_name", sequenceNameKw: read(kw), name });
   }
 
+sequence_option_logged
+  = kw:LOGGED {
+    return loc({ type: "sequence_option_logged", loggedKw: kw });
+  }
+
+sequence_option_unlogged
+  = kw:UNLOGGED {
+    return loc({ type: "sequence_option_unlogged", unloggedKw: kw });
+  }
+
 alter_sequence_stmt
   = kw:(ALTER __)
     sequenceKw:(SEQUENCE __)

src/showNode/sequence.ts

@@ -31,6 +31,8 @@ export const sequenceMap: FullTransformMap<string, AllSequenceNodes> = {
   sequence_option_owned_by: (node) => show([node.ownedByKw, node.owner]),
   sequence_option_sequence_name: (node) =>
     show([node.sequenceNameKw, node.name]),
+  sequence_option_logged: (node) => show(node.loggedKw),
+  sequence_option_unlogged: (node) => show(node.unloggedKw),
 
   // ALTER SEQUENCE
   alter_sequence_stmt: (node) =>

test/ddl/alter_table.test.ts

@@ -234,11 +234,13 @@ describe("alter table", () => {
           "ALTER COLUMN foo ADD GENERATED ALWAYS AS IDENTITY (START WITH 10 INCREMENT BY 5)"
         );
       });
-      // SEQUENCE NAME is not among the general sequence options
-      it("supports ADD GENERATED .. AS IDENTITY (SEQUENCE NAME ...)", () => {
+      // SEQUENCE NAME, LOGGED, UNLOGGED are not among the general sequence options
+      it("supports ADD GENERATED .. AS IDENTITY (special options ...)", () => {
         testAlterWc(
           "ALTER COLUMN foo ADD GENERATED ALWAYS AS IDENTITY (SEQUENCE NAME my_seq START WITH 10)"
         );
+        testAlterWc("ALTER COLUMN foo ADD GENERATED ALWAYS AS IDENTITY (LOGGED)");
+        testAlterWc("ALTER COLUMN foo ADD GENERATED ALWAYS AS IDENTITY (UNLOGGED)");
       });
 
       describe("alter identity column", () => {

test/ddl/constraints_column.test.ts

@@ -184,9 +184,11 @@ describe("column constraints", () => {
         testColConstWc("GENERATED ALWAYS AS IDENTITY (START WITH 10 INCREMENT BY 5)");
       });
 
-      // SEQUENCE NAME is not among the general sequence options
-      it("GENERATED .. AS IDENTITY (SEQUENCE NAME ...)", () => {
+      // SEQUENCE NAME, LOGGED, UNLOGGED are not among the general sequence options
+      it("GENERATED .. AS IDENTITY (special options ...)", () => {
         testColConstWc("GENERATED ALWAYS AS IDENTITY (START WITH 10 SEQUENCE NAME schm.my_seq)");
+        testColConstWc("GENERATED ALWAYS AS IDENTITY (LOGGED)");
+        testColConstWc("GENERATED ALWAYS AS IDENTITY (UNLOGGED)");
       });
     });
   });