Yes, my question is not related to this PR, the PR preserves existing behavior, and we should merge it.
It was not intuitive to me why first branch we pass `reverse` to the sort function, and in second branch we do not, so I was wondering if there was a bug in the code. Looking closer at Line 273-285, seems like the code is doing the right thing. [ Full content available at: https://github.com/apache/beam/pull/6570 ] This message was relayed via gitbox.apache.org for devnull@infra.apache.org
