[GitHub] [drill] jnturton commented on pull request #2450: DRILL-8123: Revise scan limit pushdown

[GitBox <gi...@apache.org>]

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-unsubscribe@drill.apache.org

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