Jackie-Jiang commented on pull request #7959: URL: https://github.com/apache/pinot/pull/7959#issuecomment-1022662410
Sorry for getting back and forth. I checked the behavior of MySQL, PostgresSQL and SQL Server, all of them will convert `*` to all the columns without deduplication. Let's keep this convention, and just rewrite each `*` to all columns sorted in alphabetically order to be compatible with the existing pinot behavior. The algorithm should be quite straight forward, and we can consider also rewrite the query for `*` without other columns for completeness. -- 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]
