Updates based on comments

Author: romanoff

src/parser/mod.rs

@@ -15447,10 +15447,8 @@ impl<'a> Parser<'a> {
             && self.peek_keyword_with_tokens(Keyword::SEMANTIC_VIEW, &[Token::LParen])
         {
             self.parse_semantic_view_table_factor()
-        } else if dialect_of!(self is SnowflakeDialect)
-            && self.peek_token_ref().token == Token::AtSign
-        {
-            // Snowflake stage reference: @mystage or @namespace.stage
+        } else if self.peek_token_ref().token == Token::AtSign {
+            // Stage reference: @mystage or @namespace.stage (e.g. Snowflake)
             self.parse_snowflake_stage_table_factor()
         } else {
             let name = self.parse_object_name(true)?;
@@ -15550,6 +15548,8 @@ impl<'a> Parser<'a> {
 
     /// Parse a Snowflake stage reference as a table factor.
     /// Handles syntax like: `@mystage1 (file_format => 'myformat', pattern => '...')`
+    ///
+    /// See: <https://docs.snowflake.com/en/user-guide/querying-stage>
     fn parse_snowflake_stage_table_factor(&mut self) -> Result<TableFactor, ParserError> {
         // Parse the stage name starting with @
         let name = crate::dialect::parse_snowflake_stage_name(self)?;

tests/sqlparser_snowflake.rs

@@ -4881,5 +4881,12 @@ fn test_truncate_table_if_exists() {
 
 #[test]
 fn test_select_dollar_column_from_stage() {
+    // With table function args and alias
     snowflake().verified_stmt("SELECT t.$1, t.$2 FROM @mystage1(file_format => 'myformat', pattern => '.*data.*[.]csv.gz') t");
+    // Without table function args, with alias
+    snowflake().verified_stmt("SELECT t.$1, t.$2 FROM @mystage1 t");
+    // Without table function args, without alias
+    snowflake().verified_stmt("SELECT $1, $2 FROM @mystage1");
+    // With table function args, without alias
+    snowflake().verified_stmt("SELECT $1, $2 FROM @mystage1(file_format => 'myformat')");
 }