[GitHub] [arrow-datafusion] alamb commented on a diff in pull request #6330: Ordering satisfy consider ordering equivalence of different lengths

["alamb (via GitHub)" <gi...@apache.org>]

alamb commented on code in PR #6330:
URL: https://github.com/apache/arrow-datafusion/pull/6330#discussion_r1192480677


##########
datafusion/core/tests/sqllogictests/test_files/window.slt:
##########
@@ -2379,6 +2379,31 @@ select row_number() over (rows between null preceding and current row) from (sel
 statement error DataFusion error: Error during planning: Invalid window frame: frame offsets must be non negative integers 
 select row_number() over (rows between current row and -1 following) from (select 1 a) x
 
+# This test shows that ordering satisfy considers ordering equivalences,

Review Comment:
   👍  for the explanation



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org