FrankChen021 commented on issue #10735:
URL: https://github.com/apache/druid/issues/10735#issuecomment-756784327


   I guess why `getLookupPairs` currently reload the whole table is that if 
there're records deletions in database, such a full reload could drop those 
deleted records from cache. Otherwise, if it loads records incrementally by 
tsColumn,  there must be extra code to check if records in cache have been 
deleted from database. 


----------------------------------------------------------------
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.

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org
For additional commands, e-mail: commits-h...@druid.apache.org

Reply via email to