jnturton commented on pull request #2450: URL: https://github.com/apache/drill/pull/2450#issuecomment-1028677289
Thank you for reworking these PRs @paul-rogers. It was only the timestamp fixes that I really felt should receive separate review, but these incremental PRs are great. Should I close the monolithic #2419, or switch it to draft, so that it's clear we should not merge that one? Anecdote: I had to look at a LIMIT 0 query against Parquet yesterday and saw that the Parquet reader reads a single batch of data anyway, even though I don't believe there's any need for it to. All "limit 0 shortcut" config opts that I could see were enabled. I know the present-day Parquet readers are their own animals entirely, entirely unlike EVF style readers, but I thought this anecdote might be of interest since the same LIMIT 0 case is being addressed here. -- 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: dev-unsubscr...@drill.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
