adriangb commented on PR #15475: URL: https://github.com/apache/datafusion/pull/15475#issuecomment-2761917991
> > Looks good to me. Since we're only ordering by this it shouldn't matter that we order by an integer instead of a proper timestamp, ordering is equivalent. > > Thank you very much for your reply, but I have a question: why can't the following `order by "EventTime"` be modified? It seems that they are equivalent (since the default is UTC timestamps). And both ClickHouse and duckdb do this way: [duckdb](https://github.com/ClickHouse/ClickBench/blob/main/duckdb/queries.sql#L24-L27) [clickhouse](https://github.com/ClickHouse/ClickBench/blob/main/clickhouse/queries.sql#L24-L27) I'm agreeing with you! I think this change is good 😄 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
