fix group_by all support

Author: drizk1

NEWS.md

@@ -1,4 +1,8 @@
 # TidierDB.jl updates
+##v.8.10 - 2025-08-15
+- `@arrange` can accept numeric values for columns 
+- bug fix when using `@group_by(all)` or `_by = all`
+
 ##v.8.9 - 2025-08-08
 - fixes window keyword parsing in `@mutate` that breaking nightly runs
 

Project.toml

@@ -1,7 +1,7 @@
 name = "TidierDB"
 uuid = "86993f9b-bbba-4084-97c5-ee15961ad48b"
 authors = ["Daniel Rizk <rizk.daniel.12@gmail.com> and contributors"]
-version = "0.8.9"
+version = "0.8.10"
 
 [deps]
 Arrow = "69666777-d1a9-59fb-9406-91d4454c9d45"

README.md

@@ -48,6 +48,7 @@ TidierDB.jl currently supports the following top-level macros:
 | **Aggregate Functions**          | `mean`, `minimum`, `maximum`, `std`, `sum`, `cumsum`, and nearly all aggregate sql fxns supported |
 | **Window Functions**             | `lead`, `lag`, `dense_rank`, `nth_value`, `ntile`, `rank_dense`, `row_number`, `first_value`, `last_value`, `cume_dist`, `@window_order`, `@window_frame`, or `_by`, `_order`, and `_frame` within `@mutate`                                                                                                          |
 |**Unnesting**             | `@unnest_wider` and `@unnest_longer` |
+|**Pivoting**             | `@pivot_longer` and `@pivot_wider` |
 
 `@summarize` supports any SQL aggregate function in addition to the list above. Simply write the function as written in SQL syntax and it will work.
 `@mutate` supports all builtin SQL functions as well.

docs/src/index.md

@@ -43,6 +43,7 @@ TidierDB.jl currently supports:
 | **Aggregate Functions**          | `mean`, `minimum`, `maximum`, `std`, `sum`, `cumsum`, and nearly all aggregate sql fxns supported |
 | **Window Functions**             | `lead`, `lag`, `dense_rank`, `nth_value`, `ntile`, `rank_dense`, `row_number`, `first_value`, `last_value`, `cume_dist`, `@window_order`, `@window_frame`, or `_by`, `_order`, and `_frame` within `@mutate`                                                                                                          |
 |**Unnesting**             | `@unnest_wider` and `@unnest_longer` |
+|**Pivoting**             | `@pivot_longer` and `@pivot_wider` |
 
 `@summarize` supports any SQL aggregate function in addition to the list above. Simply write the function as written in SQL syntax and it will work.   
 `@mutate` supports all builtin SQL functions as well.                                                                                                 

src/TidierDB_macros.jl

@@ -136,6 +136,8 @@ function _colref_to_string(col)
         parent_str = _colref_to_string(col.args[1])
         field_str  = string(col.args[2].value)
         return parent_str * "." * field_str
+    elseif isa(col, Integer)
+        return string(col)
     else
         throw("Unsupported column reference: $col")
     end

src/structs.jl

@@ -332,7 +332,7 @@ function finalize_query(sqlquery::SQLQuery)
      r"var\"(.*?)\"" => s"\1", r"\"\\\$" => "\"\$",  "WHERE \"" => "WHERE ", "WHERE \"NOT" => "WHERE NOT", "%')\"" =>"%\")", "NULL)\"" => "NULL)",
     "NULL))\"" => "NULL))", r"(?i)INTERVAL(\d+)([a-zA-Z]+)" => s"INTERVAL \1 \2", "SELECT SUMMARIZE " =>  "SUMMARIZE ", "\"(__(" => "(", ")__(\"" => ")"
      , "***\"" => " ", "\"***" => " ", "***" => " ", "WHERE WHERE " => "WHERE ", "WHERE  WHERE " => "WHERE ", "(__(" => "", ")__(" => "", "SELECT , CONCAT_WS" => "SELECT CONCAT_WS")
-     complete_query = replace(complete_query, ", AS " => " AS ", "OR  \"" => "OR ")
+     complete_query = replace(complete_query, ", AS " => " AS ", "OR  \"" => "OR ", "SELECT all," => "SELECT ")
     if current_sql_mode[] == postgres() || current_sql_mode[] == duckdb() || current_sql_mode[] == mysql() || current_sql_mode[] == mssql() || current_sql_mode[] == clickhouse() || current_sql_mode[] == athena() || current_sql_mode[] == gbq() || current_sql_mode[] == oracle()  || current_sql_mode[] == snowflake() || current_sql_mode[] == databricks()
         complete_query = replace(complete_query, "\"" => "'", "==" => "=")
     end