Code review comments

Author: yoavcloud

src/ast/mod.rs

@@ -7358,6 +7358,8 @@ impl Display for UtilityOption {
 /// statement to filter the results. Example from Snowflake:
 /// https://docs.snowflake.com/en/sql-reference/sql/show-tables
 #[derive(Debug, Clone, PartialEq, PartialOrd, Eq, Ord, Hash)]
+#[cfg_attr(feature = "serde", derive(Serialize, Deserialize))]
+#[cfg_attr(feature = "visitor", derive(Visit, VisitMut))]
 pub struct ShowStatementOptions {
     pub show_in: Option<ShowStatementIn>,
     pub starts_with: Option<Value>,

src/dialect/mod.rs

@@ -623,7 +623,7 @@ pub trait Dialect: Debug + Any {
         true
     }
 
-    /// Returns true if this dialect support the `LIKE 'pattern'` option in
+    /// Returns true if this dialect supports the `LIKE 'pattern'` option in
     /// a `SHOW` statement before the `IN` option
     fn supports_show_like_before_in(&self) -> bool {
         false

src/parser/mod.rs

@@ -12456,13 +12456,10 @@ impl<'a> Parser<'a> {
                 (Some(ShowStatementInParentType::Schema), None)
             }
             Some(parent_kw) => {
-                // The parent name here is still optional, for example: 
-                // SHOW TABLES IN ACCOUNT, so parsing the object name 
+                // The parent name here is still optional, for example:
+                // SHOW TABLES IN ACCOUNT, so parsing the object name
                 // may fail because the statement ends.
-                let parent_name = match self.parse_object_name(false) {
-                    Ok(n) => Some(n),
-                    _ => None,
-                };
+                let parent_name = self.maybe_parse(|p| p.parse_object_name(false))?;
                 match parent_kw {
                     Keyword::ACCOUNT => (Some(ShowStatementInParentType::Account), parent_name),
                     Keyword::DATABASE => (Some(ShowStatementInParentType::Database), parent_name),