refactor(rewrites): clarify sequence elements

ncclementi · Jun 24, 2024 · e34476b · e34476b
1 parent 12eeeed
commit e34476b
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/ibis/expr/rewrites.py b/ibis/expr/rewrites.py
@@ -311,7 +311,10 @@ def window_merge_frames(_, window):
     for sort_key in window.orderings + _.order_by:
         order_keys[sort_key.expr] = sort_key.ascending, sort_key.nulls_first
 
-    order_by = (ops.SortKey(k, *v) for k, v in order_keys.items())
+    order_by = (
+        ops.SortKey(expr, ascending=ascending, nulls_first=nulls_first)
+        for expr, (ascending, nulls_first) in order_keys.items()
+    )
     return _.copy(start=start, end=end, group_by=group_by, order_by=order_by)